Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: Support limited permissions #200

Draft
wants to merge 22 commits into
base: main
Choose a base branch
from
Draft

feat: Support limited permissions #200

wants to merge 22 commits into from

Conversation

paulgessinger
Copy link
Owner

@paulgessinger paulgessinger commented Jan 3, 2025 •
edited
Loading

Docs: https://docs.paperless-ngx.com/usage/#global-permissions

  • Test missing ui permissions setting + valid other settings
  • Refresh after permission removal should clear resources in store (see TODO in DocumentStore.swift).
  • Hide or grey-out the filter pills if we don't have view permissions
  • Communicate missing edit permissions in resource edit context
  • Prevent API error on saving new resource instances if we don't have create permissions
  • Communicate missing delete permissions in resource edit context
  • Investigate why it's possible to post documents even if no permissions
  • Check: Edit implies view (do I need to layer the tests client-side?)
  • Test document has [tags, correspondent] etc set that user does not have permissions for
  • Test document has owner set that user does not have permissions for

@paulgessinger paulgessinger added this to the v1.6.0 milestone Jan 3, 2025
@paulgessinger paulgessinger force-pushed the feat/limited-permissions branch 3 times, most recently from ad34db2 to 7f79d91 Compare January 8, 2025 21:56
@paulgessinger paulgessinger force-pushed the feat/limited-permissions branch 5 times, most recently from 6e241bf to f27585c Compare January 19, 2025 16:23
@paulgessinger paulgessinger force-pushed the main branch 4 times, most recently from 48796bd to ab1a94c Compare January 26, 2025 20:18
@paulgessinger
fix: URL Macro registration
3e4fa97
@paulgessinger
feat: Constructor Helpers for user related models
24f3116
@paulgessinger
refactor: DocumentStore gets explicit fetchUISettings
0036688
@paulgessinger
refactor: Repository.create(document:) gets return value
9e14e39
@paulgessinger
refactor: TransientRepository gets user / group management functions
52b71e5
@paulgessinger
refactor: DocumentStore suppresses PermissionsError in fetchAll
8c9fb9d
@paulgessinger
feat: DocumentList consults store permissions
7d068b9
@paulgessinger
docs: Prepare for permissions switchover
602e727
@paulgessinger
feat: DocumentStore checks perms before requesting
403682e
@paulgessinger
refactor: DocumentListViewModel checks permissions error
861b67f
@paulgessinger
refactor: Perms error output, localization
796175d
@paulgessinger
refactor: Avoid lots of errors during loading (quiet failure)
f26742d
@paulgessinger
refactor: Reorganize localized resource implementaion
ad76139
@paulgessinger
refactor: Centralize no perms and no elements view
5ccca9e
@paulgessinger
feat: Add subscript getter for permission set on user permissions
3e1b7a1
@paulgessinger
refactor: Begin reworking generic manage view with limited perms
9c7a201
@paulgessinger
refactor: additional logging in checkPermissions
7e54cd5
@paulgessinger paulgessinger force-pushed the feat/limited-permissions branch from f27585c to 41cd5d7 Compare January 26, 2025 20:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v1.6.0
Development

Successfully merging this pull request may close these issues.
1 participant
@paulgessinger