-
-
Notifications
You must be signed in to change notification settings - Fork 392
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add 'evaluating' state to proposal answers #1619
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1619 +/- ##
===========================================
- Coverage 96.99% 64.28% -32.71%
===========================================
Files 500 56 -444
Lines 8549 994 -7555
===========================================
- Hits 8292 639 -7653
- Misses 257 355 +98 |
1 similar comment
Codecov Report
@@ Coverage Diff @@
## master #1619 +/- ##
===========================================
- Coverage 96.99% 64.28% -32.71%
===========================================
Files 500 56 -444
Lines 8549 994 -7555
===========================================
- Hits 8292 639 -7653
- Misses 257 355 +98 |
Codecov Report
@@ Coverage Diff @@
## master #1619 +/- ##
==========================================
- Coverage 96.99% 96.96% -0.04%
==========================================
Files 500 500
Lines 8549 8556 +7
==========================================
+ Hits 8292 8296 +4
- Misses 257 260 +3 |
@@ -17,10 +17,13 @@ module ApplicationHelper | |||
# | |||
# Returns a String. | |||
def humanize_proposal_state(state) | |||
value = if state == "accepted" | |||
value = case state |
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.
This method smells. Can you use i18n defaults?
http://guides.rubyonrails.org/i18n.html#defaults
Something like I18n.t(value, scope: "decidim.proposals.answers", default: :not_answered)
should do the trick
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.
Beautiful
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.
You too 鉂わ笍
* Add proposal answer evaluating state * Add state evaluating to filters * Normalize locales * Add feedback
* Add proposal answer evaluating state * Add state evaluating to filters * Normalize locales * Add feedback
馃帺 What? Why?
Added
evaluating
state when answering a proposal.馃搶 Related Issues
None
馃搵 Subtasks
馃摲 Screenshots (optional)
馃懟 GIF
None