Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compress 6.1 migrations #2417

Merged
merged 3 commits into from
Jan 23, 2023
Merged

Conversation

tvdeyen
Copy link
Member

@tvdeyen tvdeyen commented Jan 21, 2023

What is this pull request for?

Remove all existing migrations and compress the
schema into one migration file that skips on existing
sites.

Checklist

@tvdeyen tvdeyen added this to the 7.0 milestone Jan 21, 2023
@tvdeyen tvdeyen force-pushed the compress-migrations branch from 9db1596 to 7021870 Compare January 22, 2023 19:38
@tvdeyen tvdeyen marked this pull request as ready for review January 22, 2023 19:39
@tvdeyen tvdeyen force-pushed the compress-migrations branch 2 times, most recently from 2a1694e to aa46ce0 Compare January 22, 2023 21:02
Remove all existing migrations and compress the
schema into one migration file that skips on existing
sites.
@tvdeyen tvdeyen force-pushed the compress-migrations branch from aa46ce0 to 4f0f565 Compare January 22, 2023 21:10
@tvdeyen tvdeyen enabled auto-merge January 22, 2023 21:10
Pass the page version into the element factory. Page version is the
direct relation to the page, the page is just a through relation and
this seems to fail with mysql in rails 6.1
Need to make sure the updated_at value is set before running
the example. Otherwise it will fail in Rails 6.1 and mysql.
@tvdeyen tvdeyen disabled auto-merge January 23, 2023 10:42
@tvdeyen tvdeyen merged commit 72194a5 into AlchemyCMS:main Jan 23, 2023
@tvdeyen tvdeyen deleted the compress-migrations branch January 23, 2023 10:42
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.

1 participant