flash-container duplicate in workflows.html.erb (small fix) - #1131
Conversation
flash-container duplicate in workflows.html.erbflash-container duplicate in workflows.html.erb (small fix)
fbacall
left a comment
There was a problem hiding this comment.
We have the following code:
https://github.com/ElixirTeSS/TeSS/blob/master/app/controllers/workflows_controller.rb#L27
https://github.com/ElixirTeSS/TeSS/blob/master/app/views/layouts/_header.html.erb#L96-L100
that was supposed to prevent this duplication occurring, but it looks like it was not applied to all cases where the workflows layout is used.
I think you should just remove the @skip_flash_messages_in_header variable and its uses
|
I just removed -> I checked where |
Summary of changes
flash-containerinworkflows.html.erbMotivation and context
Stumbled upon this duplication while logging in from the Workflows page
Screenshots
Checklist
to license it to the TeSS codebase under the
BSD license.