twilio / twilio-conversations-demo-react

Twilio Conversations Demo Web Application
MIT License
96 stars 295 forks source link

Fix Message List rendering #123

Closed josethepm closed 11 months ago

josethepm commented 11 months ago

The useEffect that was updating users was causing problems on each message send because it's supposed to be called the same amount of time each render, not ones per message.

Contributing to Twilio

All third-party contributors acknowledge that any contributions they provide will be made under the same open-source license that the open-source project is provided under.

josethepm commented 11 months ago

Noticed this is a duplicate of the work done in #119

berkus commented 11 months ago

This is a more focused PR, we can merge it faster.