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

Add a missing migration to google_sheets and google_sheets_refunds #184

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jkachel
Copy link
Contributor

@jkachel jkachel commented Feb 21, 2025

What are the relevant tickets?

mitodl/hq#6042

Description (What does it do?)

When working on mitodl/unified-ecommerce#222 I noticed there were some missing migration errors. At some point, the PKs on some of the tables within google_sheets and google_sheets_refunds switched from AutoField to BigAutoField. This adds migrations to these two apps to take care of that change.

How can this be tested?

Automated tests should pass. This is not a logic change - BigAutoField is just a bigger AutoFIeld so this should have no effect anywhere else, other than needing to run migrations after updating.

google_sheets_deferrals did not need any changes.
@annagav annagav self-requested a review February 25, 2025 16:08
@annagav annagav self-assigned this Feb 25, 2025
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.

2 participants