-
Notifications
You must be signed in to change notification settings - Fork 48
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
Allow users with report permissions to have dashboard visibility. #1959
Comments
If a user is not an administrator, they will only see the data of the records that they create. |
Thanks for the swift response @turbo124 . This means that with this UI implementation it is not possible for a user who is not an administrator to get an overview of all invoices on the dashboard, but it possible to create reports for all of them? This seems to be an inconsistent design decision. Is there a rationale for this? |
@ttytm we added the Reports/Dashboard permission levels after the view/edit/create permission levels based on feedback from users. Whilst you make a valid point about the raw data still being accessible if reports are enabled the dashboard remains out of view based on some users preference. |
Is there a chance to add a permission option that would allow viewing the data on the dashboard when it is set? |
It can be a feature request, but in this scenario , most people would just use the administrator permission. |
Yes i think so too, it'll be the usual case. It was just a case I encountered where it would be more appropriate without promoting to admin rights. Thanks again. |
@turbo124, I hope I’m not bothering you too much. Would this be considered a feature request for the main repo then? Since there is quite a gap between allowing a user to view full graphs vs promoting them to an admin, which grants access to more critical settings, allowing to create/delete organizations etc., I believe it would be appropriate to have a separate permission option for allowing users to view the graph/stats for an account. |
@ttytm sure, i've set this as a feature request. |
Repro:
The text was updated successfully, but these errors were encountered: