search

CCirbo / Little_Shop

1 stars 2 forks source link

Feature/delete item or merchant #48

Closed TDManning closed 3 weeks ago

TDManning commented 3 weeks ago

-Destroys the corresponding record and associated data including "cascading deletes" which removes associated invoices and items.

MDelarosa1993 commented 3 weeks ago

Great job on implementing these features and thoroughly testing them. This will help maintain data integrity by automatically cleaning up associated records.

litobot commented 3 weeks ago

From what I can see, this looks good and is functional. I'm a little confused about the instances work using "merchant:" instead of "merchant_id:" for the associations with invoices and items, but it is working. We confirmed it with pry.