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

Fix revamp api_keys migration #987

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

Conversation

rmartinoscar
Copy link
Contributor

@rmartinoscar rmartinoscar commented Feb 9, 2025

Mysql only allow non-null default for json since ^8.0.13
And query should be all otherwise foreach runs even if there's no api_keys in the table.

@rmartinoscar rmartinoscar requested a review from a team as a code owner February 9, 2025 23:12
@rmartinoscar rmartinoscar added the 🐛 bug Something isn't working label Feb 9, 2025
@rmartinoscar rmartinoscar marked this pull request as draft February 11, 2025 21:15
@rmartinoscar rmartinoscar force-pushed the fix/RevampApiMigration branch from d83c401 to c0b18f3 Compare March 13, 2025 00:19
@rmartinoscar rmartinoscar marked this pull request as ready for review March 13, 2025 00:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant