-
Notifications
You must be signed in to change notification settings - Fork 4
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
Make fixity dashboard viewable only by admins #907
Conversation
<ul> | ||
<li><%= link_to 'Fixity Dashboard', fixity_dashboard_path %></li> | ||
</ul> | ||
<% if current_or_guest_user.admin? %> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you just do <% if can?(:read, :fixity) %>
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also can we get a spec for viewing the link?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh yeah totally. I put that in the controller after restricting the view. will change it to use can?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
spec added. ready for re-review.
774463d
to
54f1d51
Compare
54f1d51
to
aec7cb7
Compare
@HackMasterA You said you had failing feature specs, but CI looks happy - I'll let you hit the button if it's a local issue. |
my problem was due to mismatch in versions between chrome browser and chromedriver. this fixed it https://github.com/flavorjones/chromedriver-helper#updating-to-latest-chromedriver |
fixes #899