Skip to content

Backport "Apply migration of data in surveys legacy tables automatically" to v0.22#6303

Merged
tramuntanal merged 5 commits intorelease/0.22-stablefrom
backport/fix/do_not_check_surveys_legacy_tables_for_dev_env
Jul 27, 2020
Merged

Backport "Apply migration of data in surveys legacy tables automatically" to v0.22#6303
tramuntanal merged 5 commits intorelease/0.22-stablefrom
backport/fix/do_not_check_surveys_legacy_tables_for_dev_env

Conversation

@tramuntanal
Copy link
Copy Markdown
Contributor

@tramuntanal tramuntanal commented Jul 15, 2020

🎩 What? Why?

As the original PR, this one fixes the migration of surveys' legacy tables to:

execute surveys' "legacy tables check" migration of data automatically if there is data in these tables otherwise keep on with the migrations.

📌 Related Issues

Copy link
Copy Markdown
Contributor

@Leusev Leusev 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 for me, with the development_app exception @tramuntanal

@tramuntanal
Copy link
Copy Markdown
Contributor Author

Let's wait until we merge a definitive solution into develop. Marking it as draft.

@tramuntanal tramuntanal marked this pull request as draft July 16, 2020 14:48
@tramuntanal tramuntanal changed the title Backport "Do not check surveys' legacy tables in development env" to v0.22 Backport "Apply migration of data in surveys legacy tables automatically" to v0.22 Jul 24, 2020
@tramuntanal tramuntanal marked this pull request as ready for review July 24, 2020 10:57
@tramuntanal
Copy link
Copy Markdown
Contributor Author

This PR is ready for review again

@Leusev Leusev self-requested a review July 27, 2020 08:57
end
end

def development_app?
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Where is this method used @tramuntanal ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

nowhere... I required it in a previous implementation but when reworking the PR I forgot to remove it. I've just removed it now

Copy link
Copy Markdown
Contributor

@Leusev Leusev 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 for me @tramuntanal

@tramuntanal tramuntanal merged commit 76aa8a1 into release/0.22-stable Jul 27, 2020
@tramuntanal tramuntanal deleted the backport/fix/do_not_check_surveys_legacy_tables_for_dev_env branch July 27, 2020 15:36
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