[UIIN-746] Inventory - Permissions. View Instance records which are marked as Staff suppress Created: 27/Sep/19 Updated: 18/Feb/21 Resolved: 18/Feb/21 |
|
| Status: | Closed |
| Project: | ui-inventory |
| Components: | None |
| Affects versions: | None |
| Fix versions: | None |
| Type: | Story | Priority: | P4 |
| Reporter: | Charlotte Whitt | Assignee: | Unassigned |
| Resolution: | Done | Votes: | 0 |
| Labels: | permissions | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original estimate: | Not Specified | ||
| Issue links: |
|
||||||||||||||||||||||||||||||||
| Sprint: | |||||||||||||||||||||||||||||||||
| Story Points: | 2 | ||||||||||||||||||||||||||||||||
| Development Team: | Prokopovych | ||||||||||||||||||||||||||||||||
| Tester Assignee: | Charlotte Whitt | ||||||||||||||||||||||||||||||||
| Description |
|
Purpose: In the instance record, it's possible to mark the instance record, that it should not be displayed for staff - that means typical only viewable for catalogers, who are working on a draft record, etc. To support who can view these records, we need a permission that allows users (typical catalogers) to view instance records being check marked as 'Staff suppress'. Usecase: Permission in Users to enable/disable the view of staff-suppressed instance records Scenarios:
|
| Comments |
| Comment by Zak Burke [ 17/Oct/19 ] |
|
It's important to note that they way this feature is implemented in the backend, there is no difference in the actual, server-enforced permissions between the psets "Inventory: View instances, holdings, and items" and "Inventory: View instance records being suppressed for staff". When a user views records without the latter pset, we will have to change those queries to filter out the suppressed records. |
| Comment by Marc Johnson [ 17/Oct/19 ] |
|
Zak Burke As is being discussed on
|
| Comment by Zak Burke [ 17/Oct/19 ] |
|
I was unaware of
My comment was in the context of PR #735 which implements this permission as an exact copy of ui-inventory.instance.view. IOW, because there is no back-end support for this kind of permission (an instance record is an instance record is an instance record; it doesn't matter whether its staff_suppress value is "true" or "false" any more than if its title field is "Hamlet" or "Bill and Ted's Excellent Adventure"; they're all just instance records from the point of view of permissions), the only possible implementation I considered was constructing different queries based on the presence or absence of this permission. If we're going to handle this on the backend, that's great ... but the same issue applies, it just moves the responsibility for all the work to construct such queries to the backend. |
| Comment by Khalilah Gambrell [ 17/Oct/19 ] |
|
Moving this issue to the backlog. cc: Tetyana Afanasyeva and Oleksandr Antonenko |
| Comment by Charlotte Whitt [ 18/Oct/19 ] |
|
Following up on Cate Boerema's comment in
|