Merge pull request #4546 from consul/budget-wizard-fix

Add missing attribute to budget strong params
consuldemocracy · Jun 11, 2021 · 9dc2034 · 9dc2034
2 parents eb733d1 + 7c475d4
commit 9dc2034
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/app/controllers/admin/budgets_wizard/budgets_controller.rb b/app/controllers/admin/budgets_wizard/budgets_controller.rb
@@ -37,8 +37,8 @@ def budget_params
     end
 
     def allowed_params
-      valid_attributes = [:currency_symbol, :voting_style, administrator_ids: [], valuator_ids: [],
-                          image_attributes: image_attributes]
+      valid_attributes = [:currency_symbol, :voting_style, :main_link_url, administrator_ids: [],
+                          valuator_ids: [], image_attributes: image_attributes]
 
       valid_attributes + [translation_params(Budget)]
     end

diff --git a/spec/system/admin/budgets_wizard/budgets_spec.rb b/spec/system/admin/budgets_wizard/budgets_spec.rb
@@ -7,6 +7,9 @@
       click_button "Create new budget"
       click_link "Create multiple headings budget"
 
+      fill_in "Name", with: "M30 - Summer campaign"
+      fill_in "Text on the link", with: "Participate now!"
+      fill_in "The link takes you to (add a link)", with: "https://consulproject.org"
       fill_in "Name", with: "M30 - Summer campaign"
       click_button "Continue to groups"