-
Notifications
You must be signed in to change notification settings - Fork 3
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: properly generate column names in projectKeys table #173
Conversation
drizzle/client/0000_unknown_umar.sql
Outdated
`0EncryptionKey` blob, | ||
`1EncryptionKey` blob, | ||
`2EncryptionKey` blob, | ||
`3EncryptionKey` blob, | ||
`randomEncryptionKey` blob |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Major bug exposed 🙃 fixing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed via 8c53afe
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oops I should have caught this in review
Fixes a bug introduced in #169, as I used a
for...in
instead of afor...of
loop...Also creates the drizzle files becaues I forgot to do that in the other PR. My understanding is that it should be okay to overwrite the original migration files, as opposed to creating a new migration
Referencing #165 for linking purposes