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.

Sign up for GitHub

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

Jump to bottom

Migration to Blob Storage (part 5/4) #9668

Closed
Tracked by #8661
timolegros opened this issue Oct 24, 2024 · 1 comment · Fixed by #9667
Closed
Tracked by #8661

Migration to Blob Storage (part 5/4) #9668

timolegros opened this issue Oct 24, 2024 · 1 comment · Fixed by #9667
Assignees
@timolegros
Labels
1 1-2 hour task cleanup

Comments

@timolegros
Copy link
Collaborator

timolegros commented Oct 24, 2024 •
edited
Loading

Description

  • Create a migration to add character limit to body/text columns to prevent toasting (VARCHAR(2000));
  • Update Sequelize model types

The migration to set this constraint must be separate from truncating the bodies (part 4) because truncating the bodies will be an async operation (migration).

Additional context
@timolegros timolegros added cleanup 1 1-2 hour task labels Oct 24, 2024
@timolegros timolegros self-assigned this Oct 24, 2024
This was referenced Oct 24, 2024
Closed
Merged
@timolegros
Copy link
Collaborator Author

The migration isn't working because it appears that some threads were not uploaded to Cloudflare during the last async migration (#9363). I need to investigate why that happened and get those threads uploaded before we can move forward.

@timolegros timolegros mentioned this issue Nov 1, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@timolegros timolegros

Labels
1 1-2 hour task cleanup
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Migration to limit body length to 2000 hicommonwealth/commonwealth
1 participant
@timolegros