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
Hide 'Test Digest Mail' from production #4286
Comments
Oh I think I commented on the wrong issue page @oorjitchowdhary can I take a stab at this issue? Adding environment conditional as suggested in the other thread should do it. Sent with GitHawk |
@SidharthBansal and @gauravano may be able to help more here..
|
Hi @oorjitchowdhary @uzorjchibuzor this button was added by @ViditChitkara for testing his GSoC project. I think, it should appear in front of moderators and admin only. So, please add condition accordingly. Thanks! |
I know it's for testing but you can change the button style to match other buttons. |
@gauravano so it should appear in production if the user is an admin or moderator? Sent with GitHawk |
The way it currently is, the button only appear when not in production, it doesn’t take the user status into account. Sent with GitHawk |
Yes @uzorjchibuzor it's fine to show button in production to moderators and admins. |
@uzorjchibuzor Would you like to work on this issue? |
@oorjitchowdhary Thanks, I had a PR pending that removed the button from production I am going to push a commit that takes the status of the user into account instead of the environment. I will change the style to match other buttons too while at it. Sent with GitHawk |
Oh great.. |
Made from #4280
Problem
The
Test Digest Mail
button should not appear in the production server.Code link
plots2/app/views/users/profile.html.erb
Lines 237 to 241 in 58d5ea8
The text was updated successfully, but these errors were encountered: