Skip to content
This repository has been archived by the owner on Jan 6, 2023. It is now read-only.

Permissions: Add "user" option #510

Closed
benhaynes opened this issue Oct 25, 2018 · 3 comments
Closed

Permissions: Add "user" option #510

benhaynes opened this issue Oct 25, 2018 · 3 comments
Labels
duplicate This issue or pull request already exists enhancement New feature or request question Further information is requested

Comments

@benhaynes
Copy link
Member

We have mine which is specific to the current user, but I wonder if we should have an alias called user that does the same thing. I only say this because many Admins are using this to limit permissions to specific items that they create.

eg: if an admin creates an item and "assigns" it to a user within user_created field type... then perhaps user permission and user_owner field type makes more sense?

Thoughts?

@benhaynes benhaynes added enhancement New feature or request question Further information is requested labels Oct 25, 2018
@stale
Copy link

stale bot commented Dec 24, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Dec 24, 2018
@wellingguzman
Copy link
Contributor

Ideally this should've been called owner instead of user_created. But we wanted to force to only be the user that requests it, so the meaning got lost in development.

Ideally it could be make more sense to check for permission on a field that means owner instead of author/creator. Example, An user can belongs a project now, but it can be transfer to another user (updating the field value), but the creator/author of that item could be any user.

In short I believe this should be implemented to attach a permission to an owner instead of a creator. maybe new type, or interface option? I don't want to remove this, but if we go to a new type for owner, user_created should be replaced as the field that holds the owner of an item.

@stale stale bot removed the stale label Dec 25, 2018
@benhaynes
Copy link
Member Author

That makes sense!

Still, I think we should plan on doing this in a new "dynamic rules" permissions system since that will give us a LOT of control. I will close this as a duplicate of the following ticket:

#511

@benhaynes benhaynes added the duplicate This issue or pull request already exists label Dec 26, 2018
samvasko pushed a commit to samvasko/api that referenced this issue Nov 25, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
duplicate This issue or pull request already exists enhancement New feature or request question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants