Skip to content

[SecurityBundle] Display the inherited roles of the logged-in user in the WDT #44284

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 26, 2022

Conversation

jmsche
Copy link
Contributor

@jmsche jmsche commented Nov 26, 2021

Q A
Branch? 6.1
Bug fix? no
New feature? yes
Deprecations? no
Tickets N/A
License MIT
Doc PR N/A

Hi,

As suggested on Slack by Javier it would be interesting to see inherited roles in the WDT to debug quickly.

[Edit] Aaaand the screenshot :)

screenshot-wdt

@carsonbot carsonbot added this to the 6.0 milestone Nov 26, 2021
@carsonbot carsonbot changed the title Display the inherited roles of the logged-in user in the WDT [SecurityBundle] Display the inherited roles of the logged-in user in the WDT Nov 26, 2021
@jmsche
Copy link
Contributor Author

jmsche commented Nov 26, 2021

AppVeyor failure seems unrelated

@carsonbot
Copy link

Hey!

I think @NicoHaase has recently worked with this code. Maybe they can help review this?

Cheers!

Carsonbot

@fabpot fabpot modified the milestones: 6.0, 6.1 Nov 29, 2021
@jmsche jmsche changed the base branch from 6.0 to 6.1 December 5, 2021 09:12
@jmsche jmsche force-pushed the inherited-roles-in-wdt branch from 3fa8a25 to 4c26c45 Compare March 10, 2022 13:35
@jmsche jmsche force-pushed the inherited-roles-in-wdt branch from e1b294e to eac0c8c Compare March 10, 2022 13:56
@jmsche
Copy link
Contributor Author

jmsche commented Mar 10, 2022

Just managed to rebase the 6.1 branch on my fork, so I hope this PR can get to 6.1?

@fabpot
Copy link
Member

fabpot commented Mar 26, 2022

Thank you @jmsche.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants