chingu-voyages / v36-bears-team-20

A social media website where you can find people to share activities with. | Voyage-36 | https://chingu.io/ | Twitter: https://twitter.com/ChinguCollabs
https://chingu-buddies.netlify.app/
MIT License
0 stars 0 forks source link

Chat window component (partially working) + linting with ESLint #48

Closed elliotwutingfeng closed 2 years ago

elliotwutingfeng commented 2 years ago

Adding a React component for chat between user and organizer, it currently uses dummy data stored in hooks in-lieu of the chat backend. #26 Peek 2022-01-27 23-24 Current features

It is still a work-in-progress... Upcoming features

<Need clarification on how we should proceed. Is it better to have a chat popup on the map itself, or to open up a chat window with an inbox like this?> 1*m9-IZuxS5_GXsMoLvIcVJA

For #19, I am also proposing that we use ESLint to format our JS code, it can help to keep things neat and also catch bugs. I've run the linter once in this PR, and updated the npm packages, things still run as they should from my end, let me know if anything is broken.

alvitoraidhy commented 2 years ago

Hey, I noticed that the semicolons are omitted. I think the semicolons should stay as a good practice

For example, this doesnt work without a semicolon (at least the last time I tried):

foo()
(() => doSomething())()  // error
elliotwutingfeng commented 2 years ago

Alright, I've restored semicolons in the lint files.

alvitoraidhy commented 2 years ago
alvitoraidhy commented 2 years ago

Will use this point as a base for #29

elliotwutingfeng commented 2 years ago

Chat Window Updates

alvitoraidhy commented 2 years ago

Can confirm that it works