- Merge request !9Issue #3217393: Add permission for showing only the secondary toolbar โ (Open) created by dieterholvoet
- ๐จ๐ฆCanada Nathan Tsai
Snapshot for !9 for commit 3a3c6dd7 (using the proper patch)
- ๐จ๐ฆCanada Liam Morland Ontario, CA ๐จ๐ฆ
Liam Morland โ made their first commit to this issueโs fork.
- Status changed to Needs review
9 months ago 5:19pm 8 March 2024 - ๐ฎ๐ณIndia anchal_gupta
Verified the issue on 10.x-dev.
Steps to be followed to verify the issue.
Steps1: Install and enable gin theme.
Steps2: Install and enable Gin toolbar module.
Step3: Applying the MR!9.Permissions was not there before applying the MR!9.
After The MR!9 permission for showing both the toolbar . I suggest move this issue to RTBC
Attached screenshots for reference.
manibharathi ezhimalai ravi โ made their first commit to this issueโs fork.
- Merge request !50Issue #3217393: Add secondary toolbar permission โ (Open) created by manibharathi ezhimalai ravi
Hi,
For fixing the conflicts created the separated branch from the issue fork. Changes are working as expected in the MR !50.New permissions are added for the access the toolbar.
- ๐ฌ๐งUnited Kingdom scott_euser
Thanks for the work on this!
- The latest branch with commits has merge conflicts, ie, 3217393-add-permission-for
- There seem to be multiple branches: the other branches that are unused should be hidden
- I believe we should also have an update hook so the status quo is maintained for users. Ie, if we previously expected users with 'access toolbar' to have access and all of a sudden they don't, we cause a problem updating. So we should loop through roles and grant primary & secondary permission to any role that currently has 'access toolbar'
- ๐ง๐ชBelgium dieterholvoet Brussels
@manibharathi ezhimalai ravi no need to start a new MR, you can get access to the existing one by clicking the button. I'll rebase !9 and close !50.
- ๐ง๐ชBelgium dieterholvoet Brussels
dieterholvoet โ changed the visibility of the branch 3217393-add-secondary-toolbar-permission to hidden.
- ๐ง๐ชBelgium dieterholvoet Brussels
dieterholvoet โ changed the visibility of the branch 3217391-secondary_toolbar_perm to hidden.