eScholarship / jschol

Other
10 stars 13 forks source link

Updated key constraints to cascade on update for unit id #650

Closed myucekul closed 9 months ago

myucekul commented 9 months ago

@myucekul This looks good. My only comment is that I'd like to see some inline comments linking out to the specific part of the Sequel docs that indicate how to drop constraints without dropping the foreign key itself. It's not obvious from the syntax, so a small comment could help future readers of this migration to understand what is happening.

Good point. That square bracket is super important.