Skip to content

Allow deleting links created by other users#956

Open
dallasnetprofu02-design wants to merge 1 commit into
ynput:developfrom
dallasnetprofu02-design:codex/allow-delete-foreign-links
Open

Allow deleting links created by other users#956
dallasnetprofu02-design wants to merge 1 commit into
ynput:developfrom
dallasnetprofu02-design:codex/allow-delete-foreign-links

Conversation

@dallasnetprofu02-design
Copy link
Copy Markdown

Summary

  • add an explicit links.delete_others permission so deleting links created by someone else is opt-in instead of silently broadening artist access
  • apply that permission when deleting links, while still respecting link type restrictions when the links group is enabled
  • merge the new link permission fields across access groups and cover the behavior with focused unit tests

Testing

  • $env:PYTEST_DISABLE_PLUGIN_AUTOLOAD='1'; pytest tests\\test_ayon_server_utils.py tests\\link_permissions_test.py -q

Fixes #955

@dallasnetprofu02-design
Copy link
Copy Markdown
Author

@martastain this is ready for review when you have a chance. I kept the change intentionally narrow: add an explicit opt-in permission for deleting links created by other users, preserve existing ownership checks by default, and cover the new behavior with focused unit tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

YN-0808: Permissions: User should be able to remove links created by other users

1 participant