You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
First of all thank you for this app, I followed through and learned a lot about developing chat app.
There's an issue I'd like to point out though, which resides in the useListenMessages hook. The current implementation updates messages whenever a new message comes in without checking the sender, meaning if I'm opening the conversation with A, then new messages from B will also update the chat window. The solution is to simply check if the sender matches the current conversation, and then update current messages accordingly.
Then again, it was a good run with your YT tutorial, thanks again.
The text was updated successfully, but these errors were encountered:
First of all thank you for this app, I followed through and learned a lot about developing chat app.
There's an issue I'd like to point out though, which resides in the
useListenMessages
hook. The current implementation updatesmessages
whenever a new message comes in without checking the sender, meaning if I'm opening the conversation with A, then new messages from B will also update the chat window. The solution is to simply check if the sender matches the current conversation, and then update current messages accordingly.Then again, it was a good run with your YT tutorial, thanks again.
The text was updated successfully, but these errors were encountered: