-
Notifications
You must be signed in to change notification settings - Fork 6
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
Combine feasts: 2456 -> 4474 and 2094 -> 4475 #1560
Comments
I think we can set up a management command that can do this. There's one thing I've noticed: The feast date is different for feasts 2456 and 4474, which are August 8 and August 4, respectively. Should we use the date from the old feast (2456) or the new one (4474) when merging the data? |
My understanding is that the merged feast (4474) should have the 4 August date, corresponding to the new code 14080430. |
Great point -- I could see this as being potentially useful functionality going forward. |
The command has now been run successfully on staging. Once this reaches production we can run the command again and close the issue. |
@lucasmarchd01 Thanks! I assigned you mostly so that we could see from the issue list that this was already under control -- but thanks for testing this too! |
We need to:
The text was updated successfully, but these errors were encountered: