The 'Notify Newly Added Users via Email' Feature Remains Accessible on UI/UX despite Permission Scheme Configuration Restrictions
Description
The "Notify Newly Added Users via Email" option remains accessible for selected users, even when "Send share by email" permission has been disabled. Please note that no email is actually sent out however this inconsistency will lead to confusion for the user.
Step to Reproduce
- Login as an admin
- Enable permission schemes (Global settings)
- Navigate to the Permission Schemes
- Remove all access from the ”Send share by email” permission
- On any page create a share
- Add selected users
- Notice the “Notify Newly Added Users via Email“ remains available
Actual Result
The "Notify" option remains accessible, even though the user's "Send share by email" permission has been disabled.
Expected Result
The "Notify" option should be removed (or disabled) for users whose "Send share by email" permission has been disabled.
Activity
Show:
@Michał Błaszczykowski Improvement implemented - QA environment.
Hello @Michał Błaszczykowski,
Please merge code to dev branch.
This is the best moment to add more information that can be helpful to prepare release notes.
Can you prepare short overview of change that can be used in release notes?
Please provide short GIF that showcase feature.
If GIF make no sense, can you provide image that highlights feature that can be used in release notes (cropped & annotated)?
Hello @Michał Błaszczykowski ]
This is the best moment to add more information that can be helpful for tester.
What areas are affected?
What are potential edge cases?
Was it checked for XSS problems?
Does change affect security, is new data exposed?
Please attach - Before / After screenshot if possible.
Hello @Michał Błaszczykowski,
Change was reviewed and approved.
Task is ready to be deployed to QA.
Once it is deployed to QA please move ticket to "To Test"
Thank you!
Hello @Krzysztof Bogdan,
Task is ready for review.
@Michał Błaszczykowski please make sure reviewer
have easy access to contend to be reviewed.
If this is code change. Please make sure PR is created.
If this is new documentation, blogpost, etc. Please provide link to page.
Hello @Krzysztof Bogdan
This is the best moment to add more description, wire-frames, comments.
Why we doing this?
What is expected result?