#3288 - Enabled 'admin-div' only for Site Administrator #3308
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.
References
Description
This PR introduces a change to the CreateCommunityParentSelectorComponent to display the admin-div section only to users with Site Administrator privileges.
Instructions for Reviewers
The key changes include:
New
isAdmin$
Observable: Added an isAdmin$ observable to the CreateCommunityParentSelectorComponent to track admin status.Conditional Rendering: Added a
<div>
withdata-test="admin-div"
in the component's template, which is displayed only ifisAdmin$
emitstrue
.Updated Tests: Adjusted the unit tests to check for the presence of the
admin-div
based on the admin status.How to test:
This will confirm that the
admin-div
is correctly shown or hidden based on the user's admin status.Checklist
main
branch of code (unless it is a backport or is fixing an issue specific to an older branch).yarn lint
yarn check-circ-deps
)package.json
), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.