Skip to content

Admin configuration of amendments by step#5178

Merged
oriolgual merged 11 commits intodecidim:masterfrom
CodiTramuntana:feat/amendable_by_step_config
Jun 17, 2019
Merged

Admin configuration of amendments by step#5178
oriolgual merged 11 commits intodecidim:masterfrom
CodiTramuntana:feat/amendable_by_step_config

Conversation

@agustibr
Copy link
Copy Markdown
Contributor

@agustibr agustibr commented Jun 5, 2019

🎩 What? Why?

Adds ability to admin to configure amendments by step:

  • general config: enable/disable amendments
  • step config:
    • enable/disable amendment creation
    • enable/disable amendment reaction
    • enable/disable rejected emendation promotion to proposal

Cell to show links to/from rejected emendation with proposal.

📌 Related Issues

📋 Subtasks

  • Add CHANGELOG entry
  • Add documentation regarding the feature
  • Add/modify seeds
  • Add tests
  • Another subtask

📷 Screenshots (optional)

Description

@agustibr agustibr force-pushed the feat/amendable_by_step_config branch 2 times, most recently from 944f4f9 to 086275a Compare June 11, 2019 15:49
@agustibr agustibr force-pushed the feat/amendable_by_step_config branch from 086275a to 25c88e2 Compare June 12, 2019 07:28
@agustibr agustibr marked this pull request as ready for review June 14, 2019 08:33
@agustibr agustibr requested a review from a team as a code owner June 14, 2019 08:33
@agustibr
Copy link
Copy Markdown
Contributor Author

@aitorlb great work!

I refactored a spec in the last commit but realized after fixing a rubocop
warning that the error was not being raised anymore.

The error is a regression that was fixed in decidim#5179, which updated the component
setting :participatory_texts_enabled from true to false when the checkbox
was being disabled from the frontend.

I disabled the rubocop warning and made sure the error is catched now, testing
it in the branch that introduced the regression.
oriolgual
oriolgual previously approved these changes Jun 14, 2019
@aitorlb
Copy link
Copy Markdown
Contributor

aitorlb commented Jun 15, 2019

@oriolgual the diff of the amend_proposal_spec file is kind of a mess, so I attach an image to show how the spec reads when executing it with --format documentation:

amend_proposal_spec

@oriolgual oriolgual merged commit 1b99136 into decidim:master Jun 17, 2019
@agustibr
Copy link
Copy Markdown
Contributor Author

Y'all are great! 😄 ⚙️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants