spencerlepine / woofer

Dating app for pets - a full stack MERN project. Complete with CI/CD pipeline w/ Jest, GitHub Actions, Docker Hub, and AWS EC2

https://youtu.be/aiJhCoZRc78

2 stars 1 forks source link

fixing the Match Making System #95

Closed spencerlepine closed 2 years ago

spencerlepine commented 2 years ago

Pull Request #94

❌ Broken match system

User should be able to swipe YES on a user and get a generated chat room to join.

Flowchart

MatchSystemDiagram

spencerlepine commented 2 years ago

94 - wrote integration tests to test the API request [98793f2]

Identified TWO errors:

Zipcode Fix

Everytime a user was added to a zipcode (as an object key/value pair), the object variable assignment was done incorrectly. Previous userIds were erased, and the zipcode always had ONE user, not a list.

Chat Ids Fix

When users swipe on eachother, the chat Ids were not being added to their profiles