search

nickslick03 / messiah-meal-planner

1 stars 0 forks source link

Duplicate meals in day editor are removed as a pair #25

Closed nickslick03 closed 1 month ago

nickslick03 commented 1 month ago

mmp_issue_1

As the GIF shows, when a meal is added as a duplicate at least twice to the same day, when that meal is removed it removes the duplicate as well.

cdrice26 commented 1 month ago

Ah good catch. I assigned IDs when adding to the meal queue to prevent this issue in the meal queue, but I forgot to reassign them when adding to the day editors. I pushed a fix to the state-management branch and created pull request #26 to bring this patch into development. It says there's a conflict, but it's just two different imports. All we need to do is when we are ready to merge, use the web editor to delete the diff lines and keep both imports.