-
Notifications
You must be signed in to change notification settings - Fork 14
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.
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
Permissions for non-admin users #24
Comments
Hi @BeckeBauer, thanks for your feature request. Should be no problem to implement a set of permissions and roles. I've thought about permissions when implementing this plugin, but for simplicity reasons I used the super admin role. I think there should be more than one permission:
Maybe 2 + 4 can be merged into one permission. If any of these permissions are given, the user will see the navigation/menu button. A new role ROLE_SHARED_PROJECT_TIMESHEETS_ADMIN could be added to combine all permissions. Subsets of the permissions can be created using https://www.kimai.org/documentation/permissions.html Is the proposed solution sufficient for your needs? Do you agree with the permissions and roles? Any other ideas or improvements? 🙂 Edit: by default the ROLE_SUPER_ADMIN should be assigned to all the permissions as well (backward compatibility). |
Sounds great 👍 As regards the introduction of a new role: not sure if it were necessary if the 3(or 4) permissions can be be assigned to the existing roles (a new column for a role which is used only for the plugin might reduce the visibility of role's overview which is already not very clear - I hope I'm not missing the point here) |
It would be great if non admin users could use the plugin as well. I have contacted the developer of kimai and his response was as follow:
Maybe it helps to amend the plugin?
The text was updated successfully, but these errors were encountered: