-
Notifications
You must be signed in to change notification settings - Fork 121
feat: migrate default postgres schema #2381
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
Conversation
|
Vercel deployment URL: https://stacks-blockchain-phbdfydyl-hirosystems.vercel.app 🚀 |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
|
Thanks for this PR @rmottley-hiro ! If we merge this change into the |
c93a57a to
b3329df
Compare
Sounds good! 👍 |
b3329df to
68b5ae6
Compare
68b5ae6 to
f4ca978
Compare
f4ca978 to
bd9bd74
Compare
bd9bd74 to
47de8e8
Compare
rafaelcr
left a comment
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.
![]()
|
@rmottley-hiro there's just one broken test but feel free to merge after that is fixed 👍 thanks! |
- Migrate schema from public to stacks-blockchain-api. BREAKING CHANGE: This change will require resyncing from genesis
47de8e8 to
2cc2459
Compare
Closes #2373
BREAKING CHANGE: This change can be made without a full rebuild from genesis. However, it is considered breaking in the interest of data safety
Testing information
Testing is recommended. Ensure
PG_OLD_SCHEMAmatches the current schema name in your database.PG_SCHEMAis pre-set, but can be modified as the user wishes. UncommentPG_MIGRATE_SCHEMA_ON_STARTUP(app will fail to start until this is set as a value to prevent accidental migration)Checklist
npm run testpasses