-
-
Notifications
You must be signed in to change notification settings - Fork 632
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 user role to view history #890
Allow user role to view history #890
Conversation
This pull request introduces 1 alert when merging 7727094 into 4c19f95 - view on LGTM.com new alerts:
|
Can you merge the changes from master to your branch? Sorry I was so slow with the PRs :( |
no problem, I've merged the changes into master |
It seems like you forgot pushing your changes to resolve the conflicting files here |
This pull request introduces 2 alerts and fixes 1 when merging c698889 into 44c4531 - view on LGTM.com new alerts:
fixed alerts:
|
This pull request introduces 1 alert and fixes 1 when merging 1fb2a13 into 44c4531 - view on LGTM.com new alerts:
fixed alerts:
|
This PR a includes a configuration option to allow the user role to view history records relating to the domains they have access to.