Skip to content
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

Clarify the meaning of max "votable" headings #4126

Merged
merged 2 commits into from
Sep 9, 2020
Merged

Conversation

javierm
Copy link
Member

@javierm javierm commented Sep 9, 2020

Background

We've got two different phases where citizens can participate during a budget: the selecting projects phase, where users support projects so they can enter the voting phase, and the mentioned voting phase. We were using a text saying it affects the number of headings where users can vote, but it only affected the selecting projects phase and not the voting phase.

Objectives

Make it clear the maximum number of headings only applies to the selecting projects phase.

Visual Changes

Before these changes

"Maximum number of headings in which a user can vote" is used as label

After these changes

"Maximum number of headings in which a user can vote" is used as label and there's a hint clarifying in only applies to the selecting projects phase

@javierm javierm self-assigned this Sep 9, 2020
@javierm javierm added this to Reviewing in Consul Democracy via automation Sep 9, 2020
spec/lib/consul_form_builder_spec.rb Outdated Show resolved Hide resolved
spec/lib/consul_form_builder_spec.rb Outdated Show resolved Hide resolved
Just like we allow them on input fields.
@javierm javierm force-pushed the max_headings_text branch 2 times, most recently from 94bd31a to 49927ae Compare September 9, 2020 10:52
The word "vote" is confusing because this option does not apply to the
voting phase, but to the selecting projects phase.
Consul Democracy automation moved this from Reviewing to Testing Sep 9, 2020
@javierm javierm merged commit a398f4a into master Sep 9, 2020
Consul Democracy automation moved this from Testing to Release 1.2.0 Sep 9, 2020
@javierm javierm deleted the max_headings_text branch September 9, 2020 16:57
@javierm javierm removed the 1.2 label Sep 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants