search

80O / PlusEMU

37 stars 27 forks source link

Add UserWardrobeManager #106

Closed chrismpettyjohn closed 2 years ago

chrismpettyjohn commented 2 years ago

Story As a Developer, I want a reusable service for adding and removing looks from the user wardrobe.

Changes

  1. Created UserWardrobeManager
  2. Updated SaveWardrobeOutfitEvent to use the new UserWardrobeManager

Issues Related to #4

Testing I validated locally that users could create outfits and save them in various slots, or overwrite existing slots.

Video Nitro

chrismpettyjohn commented 2 years ago

Upon testing, it looks like there's a few issues with the DB queries to work out. Will update sometime tomorrow

chrismpettyjohn commented 2 years ago

I resolved the test issues and have verified users can successfully interact with their wardrobe. Apart of these changes, are a new database migration that prevents multiple rows being added for the same user, slot and gender combination

Nitro

80O commented 2 years ago

I think unique constraint should be over user_id and slot_id. Cant insert multiple looks for different genders in the same slot I think?

chrismpettyjohn commented 2 years ago

I think unique constraint should be over user_id and slot_id. Cant insert multiple looks for different genders in the same slot I think?

Changes pushed

chrismpettyjohn commented 2 years ago

Closing as resolved with more fn by above PR