Update db/schema.rb to match migrations#10815
Conversation
[skip changelog]
| # These are extensions that must be enabled in order to support this database | ||
| enable_extension "citext" | ||
| enable_extension "pg_stat_statements" | ||
| enable_extension "pgcrypto" |
There was a problem hiding this comment.
I think this is left over from https://github.com/18F/identity-idp/pull/5275/files#diff-51a7a6164b57d05d4ec51d2c66b5fc9abb235f36f41ce2188cf9ab13c15bd256R4 since the GoodJob migration
was originally in this folder.
[skip changelog]
| t.boolean "ready_for_status_check", default: false | ||
| t.datetime "notification_sent_at", comment: "The time a notification was sent" | ||
| t.datetime "last_batch_claimed_at" | ||
| t.string "sponsor_id", comment: "The identification number for USPS to recognize us and our flow (ex: Enhanced IPP)" |
There was a problem hiding this comment.
I think this comment was intentionally added based on review feedback, so the sync-up should probably have been to include the comment in the migration itself.
There was a problem hiding this comment.
The comment won't appear in the database in environments that have already run this migration, and I'm not sure how often we update migrations that have already been run (though a comment is maybe not over that line)
There was a problem hiding this comment.
Good point. I suppose if it's important to have the comment, a separate migration just to add the comment might be the way to go.
3 participants
🛠 Summary of changes
#10759 included
db/schema.rbchanges that differed slightly from the result of running migrations.