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
Add an option to hide the "Can I vote?" page #9191
Conversation
28a6d7e
to
4c83e84
Compare
Fixes #9124 |
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.
LGTM!
I went ahead and fixed the merge conflict with the translations file, so on my part this is good to go.
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.
Let's change this to "show_check_census", as "Can I vote?" is only the string menu/title and can/will change but check_census
is the name of the action in the controller, the command, etc.
decidim-elections/app/commands/decidim/votings/admin/create_voting.rb
Outdated
Show resolved
Hide resolved
decidim-elections/app/commands/decidim/votings/admin/update_voting.rb
Outdated
Show resolved
Hide resolved
decidim-elections/app/forms/decidim/votings/admin/voting_form.rb
Outdated
Show resolved
Hide resolved
decidim-elections/app/views/decidim/votings/admin/votings/_form.html.erb
Outdated
Show resolved
Hide resolved
decidim-elections/spec/commands/decidim/votings/admin/create_voting_spec.rb
Outdated
Show resolved
Hide resolved
decidim-elections/spec/commands/decidim/votings/admin/create_voting_spec.rb
Outdated
Show resolved
Hide resolved
decidim-elections/spec/commands/decidim/votings/admin/create_voting_spec.rb
Outdated
Show resolved
Hide resolved
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.
Let's change this to "show_check_census", as "Can I vote?" is only the string menu/title and can/will change but check_census
is the name of the action in the controller, the command, etc.
0c92c2c
to
6a13ebf
Compare
Done @andreslucena |
Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
6a13ebf
to
8c66ec7
Compare
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.
👍🏽 Thanks!
* Add option to hide the "Can I vote?" link * Add missing translations for votings form * Rename to `show_check_census` Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com> Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
* Add option to hide the "Can I vote?" link * Add missing translations for votings form * Rename to `show_check_census` Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com> Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
* Add option to hide the "Can I vote?" link * Add missing translations for votings form * Rename to `show_check_census` Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com> Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
🎩 What? Why?
Adds an option to the votings form to be able to hide the "Can I vote?" page, this was requested after the running the simulation on the evote module.
Also adds missing translations for the votings form.
📷 Screenshots