Add location manager role #6566
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ThiefMaster
reviewed
Oct 1, 2024
indico/migrations/versions/20240911_1515_b212f6d17229_add_location_principals_table.py
Outdated
Show resolved
Hide resolved
indico/migrations/versions/20240911_1515_b212f6d17229_add_location_principals_table.py
Outdated
Show resolved
Hide resolved
ThiefMaster
reviewed
Oct 1, 2024
SegiNyn
force-pushed
the
add-location-manager
branch
3 times, most recently
from
ae5b407 to
49adb1a
Compare
ThiefMaster
force-pushed
the
add-location-manager
branch
from
49adb1a to
49fba84
Compare
ThiefMaster
reviewed
Mar 5, 2025
ThiefMaster
force-pushed
the
add-location-manager
branch
from
5db14cd to
1e353c9
Compare
SegiNyn
force-pushed
the
add-location-manager
branch
from
622ce6f to
f2f5041
Compare
Contributor
Author
|
Hello @ThiefMaster, I've updated the PR to allow permissions assigned to the location manager (prebook, book, moderate) to be applicable to the rooms in the location. And if the manage permission is assigned then the location manager will be able to see a new section "My locations" where they can further manage rooms |
ThiefMaster
reviewed
Mar 6, 2025
ThiefMaster
force-pushed
the
add-location-manager
branch
from
f2f5041 to
a84ae62
Compare
ThiefMaster
force-pushed
the
add-location-manager
branch
from
7f8b1a7 to
f752e2c
Compare
SegiNyn
added a commit
to UNOG-Indico/indico-core
that referenced
this pull request
Mar 10, 2025
Add new Location ACL, and allow location managers to manage their location and its rooms without being RB admins. Co-authored-by: Adrian Moennich <[email protected]>
OmeGak
pushed a commit
to UNOG-Indico/indico-core
that referenced
this pull request
Mar 12, 2025
Add new Location ACL, and allow location managers to manage their location and its rooms without being RB admins. Co-authored-by: Adrian Moennich <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pr is to add a permissions at the location-level in the RB to determine who can manage it and giving access to the Location part of the RB.