Fix text confirming investment heading support

We accidentally removed the `count` option in commit 55fb14ac, which
made the translation return a hash.

The test is a bit hacky, which makes me think changing the user
interface would probably be a better solution.
This commit is contained in:
Javi Martín
2019-07-08 15:33:52 +02:00
parent de50317d25
commit 2f10005739
3 changed files with 9 additions and 2 deletions

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 %>

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"

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