[UITEN-166] Hide or fix the permissions related to Tenant Settings Created: 09/Feb/21 Updated: 12/Mar/21 Resolved: 12/Mar/21 |
|
| Status: | Closed |
| Project: | ui-tenant-settings |
| Components: | None |
| Affects versions: | None |
| Fix versions: | None |
| Type: | Bug | Priority: | P2 |
| Reporter: | Ann-Marie Breaux (Inactive) | Assignee: | Michal Kuklis |
| Resolution: | Done | Votes: | 0 |
| Labels: | permissions | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original estimate: | Not Specified | ||
| Attachments: |
|
||||||||||||||||||||||||||||||||
| Issue links: |
|
||||||||||||||||||||||||||||||||
| Sprint: | Prokopovych - Sprint 109 | ||||||||||||||||||||||||||||||||
| Development Team: | Prokopovych | ||||||||||||||||||||||||||||||||
| Release: | R1 2021 | ||||||||||||||||||||||||||||||||
| Tester Assignee: | Ann-Marie Breaux (Inactive) | ||||||||||||||||||||||||||||||||
| Description |
|
Purpose: To hide or update the UI permissions related to Tenant Settings As a user assigning permissions to other users Scenarios:
|
| Comments |
| Comment by Holly Mistlebauer [ 08/Mar/21 ] |
|
Question from Zak on Slack... "This likely involves translation updates. Normally we only touch the en.js file, but here I think I would update en_US.js and all the other en*.js_ files. Are there any non-English deployments that will use this release? If so, we need to coordinate with @peter-murray to get this change out to Lokalise and then back into GitHub as well." Answer from Peter Murray on Slack... "Anything that is done in the English language to en.js is automatically copied to en_US.js, so I don't think they need to do anything. (If they need to ask for an emergency push from Lokalise to GitHub, they can do that. It happens occasionally.)" |
| Comment by Holly Mistlebauer [ 08/Mar/21 ] |
|
Zak Burke: I have added the comment above in response to your question about coordinating with Peter, etc. Please contact Peter if you have further questions. Thanks... |
| Comment by Zak Burke [ 08/Mar/21 ] |
Holly Mistlebauer, This is a bit involved as visibility of each setting (Address, Key bindings, etc.) is determined by specific permissions. Thus, the ability to "view ... all settings" actually entails several pieces of work:
Only the last is significant, but it may be quite significant because some of these pages were not built with separate "read" and "update" views in mind (e.g. key bindings, language and localization, preferred plugins, SSO...) |
| Comment by Michal Kuklis [ 10/Mar/21 ] |
|
The work on this should be now done without one requirement from scenario 1:
This has been moved to a new JIRA: https://folio-org.atlassian.net/browse/UITEN-167 |
| Comment by Ann-Marie Breaux (Inactive) [ 10/Mar/21 ] |
|
Thanks Michal Kuklis Should
|
| Comment by Yelyzaveta Khokhlova [ 11/Mar/21 ] |
|
Ann-Marie Breaux, should
|
| Comment by Ann-Marie Breaux (Inactive) [ 11/Mar/21 ] |
|
Hi Yelyzaveta Khokhlova If it's possible, that would be great. It sounds like
|
| Comment by Ann-Marie Breaux (Inactive) [ 12/Mar/21 ] |
|
Hi Michal Kuklis and Magda Zacharska tested on folio-snapshot. All is good. The settings screen is stil bare, but that won't change until
|
| Comment by Ann-Marie Breaux (Inactive) [ 12/Mar/21 ] |
|
Magda Zacharska or Yelyzaveta Khokhlova we still need to confirm which team will do
|