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

Store custom sources in SQL instead of blobs #1693

Merged
merged 3 commits into from
Aug 1, 2024

Conversation

andchiind
Copy link
Contributor

Closes #1577

@andchiind andchiind added backend Backend related functionality improvement Improvement to existing functionality database-change Will require migration labels Jul 25, 2024
@andchiind andchiind self-assigned this Jul 25, 2024
Copy link

🔔 Changes in database folder detected 🔔
Do these changes require adding new migrations? 🤔 In that case follow these steps.
If you are uncertain, ask a database admin on the team 😄

@andchiind andchiind force-pushed the 1577-remove-blob branch 3 times, most recently from 6dccfc2 to 806d153 Compare July 26, 2024 08:59
Copy link

🔔 Migrations changes detected 🔔
📣 Remember to comment "/UpdateDatabase" after review approval for migrations to take effect!

@andchiind andchiind force-pushed the 1577-remove-blob branch 2 times, most recently from 1926ead to b280e05 Compare July 26, 2024 09:06
@andchiind andchiind marked this pull request as ready for review July 26, 2024 10:13
Copy link
Contributor

@betaniat betaniat left a comment

Choose a reason for hiding this comment

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

LGTM!

@andchiind
Copy link
Contributor Author

/UpdateDatabase

Copy link

github-actions bot commented Aug 1, 2024

👀 Running migration command... 👀

Copy link

github-actions bot commented Aug 1, 2024

✨ Successfully ran migration command! ✨

@andchiind andchiind merged commit 86ef1f4 into equinor:main Aug 1, 2024
12 checks passed
@andchiind andchiind deleted the 1577-remove-blob branch August 1, 2024 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Backend related functionality database-change Will require migration improvement Improvement to existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Store custom missions in SQL table instead of blob storage
2 participants