Skip to content

Improve proposals import options#7669

Merged
mrcasals merged 5 commits intodecidim:developfrom
Platoniq:feature/improve-proposals-import-options
Apr 19, 2021
Merged

Improve proposals import options#7669
mrcasals merged 5 commits intodecidim:developfrom
Platoniq:feature/improve-proposals-import-options

Conversation

@verarojman
Copy link
Copy Markdown
Contributor

@verarojman verarojman commented Mar 22, 2021

🎩 What? Why?

This PR adds these two options when importing proposals from one component to another:

  • Import the status of the proposal and the answer
  • Import the proposals of a certain scope

📌 Related Issues

Testing

decidim-proposals/spec/commands/decidim/proposals/admin/import_proposals_spec.rb

📋 Checklist

  • ✔️ DO check open PR's to avoid duplicates.
  • ✔️ DO keep pull requests small so they can be easily reviewed.
  • ✔️ DO build locally before pushing.
  • ✔️ DO make sure tests pass.
  • ✔️ DO add link to MetaDecidim if it's a new feature.
  • AVOID breaking the continuous integration build.
  • AVOID making significant changes to the overall architecture.

📷 Screenshots

Proposals import options

Proposals import scope picker

♥️ Thank you!

@verarojman verarojman marked this pull request as ready for review March 22, 2021 18:26
@mrcasals
Copy link
Copy Markdown
Contributor

I know it's out of your scope, but beware of #7608!

@verarojman
Copy link
Copy Markdown
Contributor Author

verarojman commented Apr 6, 2021

I know it's out of your scope, but beware of #7608!

Thanks for the heads-up :) Could we get it merged in spite of the bug?

@microstudi
Copy link
Copy Markdown
Contributor

@mrcasals is there anything we should do to get this merged? From what I see #7608 is a different beast and this improvements to proposal imports shouldn't make much of a difference to whatever solution will be implemented.

Thanks!

@mrcasals mrcasals merged commit 69b04e7 into decidim:develop Apr 19, 2021
@mrcasals
Copy link
Copy Markdown
Contributor

@verarojman @microstudi sorry, I totally forgot about this one. Merged, thanks!!

entantoencuanto added a commit that referenced this pull request Apr 19, 2021
* develop:
  New Crowdin updates (#7850)
  Accessibility tool for development environments (#7810)
  Attempt to fix puffing-billy runtime error (#7853)
  Dynamically get all participatory space role tables for the `visible_meeting_for` query (#7855)
  Fix joining a meetings that the user already follows (#7854)
  Improve proposals import options (#7669)
  Fix proposal form attachment errors (#7856)
  New Crowdin updates (#7844)
  Fix branch name on generators (#7849)
  Conditionally render questions in election booth (#7822)
  Fix node version for version managers (#7848)
  Migrate to Webpacker (#7464)
microstudi pushed a commit to Platoniq/decidim that referenced this pull request Apr 22, 2021
@microstudi microstudi mentioned this pull request Apr 26, 2021
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants