Skip to content

Fix issues with move proposal fields to i18n#6838

Merged
ivan-mr merged 1 commit intodecidim:developfrom
mainio:fix/broken-move-proposal-fields-migration
Nov 10, 2020
Merged

Fix issues with move proposal fields to i18n#6838
ivan-mr merged 1 commit intodecidim:developfrom
mainio:fix/broken-move-proposal-fields-migration

Conversation

@ahukkanen
Copy link
Copy Markdown
Contributor

@ahukkanen ahukkanen commented Nov 10, 2020

🎩 What? Why?

There are few noticed issues with the move proposal fields to i18n migration that seem to repeat in most older instances when trying to run the migrations:

  • There may not be an existing author for the proposal (e.g. deleted users)
  • The old proposals may not follow all the validation criteria which causes the whole migration to fail

📌 Related Issues

Testing

Try to run the migration in an instance with a proposal left by a deleted user or an instance with proposals in the database that do not follow the validation criteria.

📋 Checklist

  • CONSIDER adding a unit test if your PR resolves an issue.
  • ✔️ 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 make sure any new changes are documented in docs/.
  • ✔️ DO add and modify seeds if necessary.
  • ✔️ DO add CHANGELOG upgrade notes if required.
  • ✔️ DO add to GraphQL API if there are new public fields.
  • ✔️ 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.

@PierreMesure
Copy link
Copy Markdown
Contributor

I got this issue! I don't think people should (can?) upgrade to v0.23 before this is fixed!

@ivan-mr
Copy link
Copy Markdown
Contributor

ivan-mr commented Nov 10, 2020

I got this issue! I don't think people should (can?) upgrade to v0.23 before this is fixed!

I think the intention of this PR is to be able to later perform a backport to version 0.22, right? As you say the correction should be made before advancing to version 0.23 and the PR to which is referenced to fix I think is earlier than version 0.23

Copy link
Copy Markdown
Contributor

@ivan-mr ivan-mr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Thanks @ahukkanen !

@ivan-mr ivan-mr merged commit 3b1aaaf into decidim:develop Nov 10, 2020
@ahukkanen ahukkanen deleted the fix/broken-move-proposal-fields-migration branch November 11, 2020 07:05
@ahukkanen
Copy link
Copy Markdown
Contributor Author

@ivan-mr Even though #6285 merged before the 0.22 release date, I don't see this particular migration being part of the official 0.22 gem. I think the 0.22 version was cut from develop on 12 June based on this commit efc4324. #6285 was merged at the end of July.

My plan was to backport only to 0.23 as it's already released. Also, according to the security document, 0.22 is no longer officially supported.

Before backporting, could you @ivan-mr please check also #6839. It is related to this and needs to be backported as well.

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

Labels

in-review module: proposals type: fix PRs that implement a fix for a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants