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

rebuild-test-project-fixture: Hardcode migration time #8656

Merged

Conversation

Tobbe
Copy link
Member

@Tobbe Tobbe commented Jun 18, 2023

We had code in place to limit churn on migrations when regenerating the test project. But timezones messed it up sometimes. This PR tries to solve that problem.

This is an example of what it could look like. Notice 19:00:00 vs 13:00:00

image

Now it's hardcoded to 12:00:00, which I think was the intent of the original code

@Tobbe Tobbe added the release:chore This PR is a chore (means nothing for users) label Jun 18, 2023
Copy link
Collaborator

@Josh-Walker-GM Josh-Walker-GM left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this!

@Tobbe Tobbe enabled auto-merge (squash) June 18, 2023 22:41
@Tobbe Tobbe merged commit 2e355f3 into redwoodjs:main Jun 18, 2023
28 checks passed
@redwoodjs-bot redwoodjs-bot bot added this to the next-release milestone Jun 18, 2023
@Tobbe Tobbe deleted the tobbe-test-project-fixture-migration-dates branch June 19, 2023 05:48
@jtoar jtoar modified the milestones: next-release, v6.0.0 Jun 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:chore This PR is a chore (means nothing for users)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants