-
Notifications
You must be signed in to change notification settings - Fork 13
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
Fyst 1515 add prior year access link button to homepage #5296
base: FYST-1466-prior-year-access-mvp
Are you sure you want to change the base?
Fyst 1515 add prior year access link button to homepage #5296
Conversation
Heroku app: https://gyr-review-app-5296-1489fbdcf0f6.herokuapp.com/ |
end | ||
|
||
def down | ||
raise ActiveRecord::IrreversibleMigration |
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.
given this is a migration of existing data, i believe it's a fair assumption to purge the attached pdf and truncate the tables.
db90448
to
36a8be4
Compare
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.
thanks for setting up the initial flow 🎉
Link to pivotal/JIRA issue
Is PM acceptance required? (delete one)
Reminder: merge main into this branch and get green tests before merging to main
What was done?
How to test?
Screenshots (for visual changes)