consul/grecia
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #3656 from consul/fix_alert_text

Browse Source 
Fix text confirming investment heading support
This commit is contained in:
Javier Martín
2019-09-11 15:34:50 +02:00
committed by GitHub
parent f362939eb3 2f10005739
commit 88afeda085
3 changed files with 9 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/views/budgets/investments/_votes.html.erb
View File

@@ -20,7 +20,7 @@
method: "post",
remote: (display_support_alert?(investment) ? false : true),
data: (display_support_alert?(investment) ? {
confirm: t("budgets.investments.investment.confirm_group") } : nil),
confirm: t("budgets.investments.investment.confirm_group", count: investment.group.max_votable_headings) } : nil),
"aria-hidden" => css_for_aria_hidden(reason) do %>
<%= t("budgets.investments.investment.give_support") %>
<% end %>

2
config/locales/en/budgets.yml
View File

@@ -137,7 +137,7 @@ en:
support_title: Support this project
confirm_group:
one: "You can only support investments in %{count} district. If you continue you cannot change the election of your district. Are you sure?"
other: "You can only support investments in %{count} district. If you continue you cannot change the election of your district. Are you sure?"
other: "You can only support investments in %{count} districts. If you continue you cannot change the election of your district. Are you sure?"
supports:
one: 1 support
other: "%{count} supports"

7
spec/features/budgets/votes_spec.rb
View File

@@ -191,6 +191,13 @@ describe "Votes" do
"Share it!"
end
scenario "Confirm message shows the right text", :js do
visit budget_investments_path(budget, heading_id: new_york.id)
find(".in-favor a").click
expect(page.driver.send(:find_modal).text).to match "You can only support investments in 2 districts."
end
end
end
end