mirror of
https://github.com/coder/coder.git
synced 2026-09-24 15:04:27 +08:00
Bruno Quaresma
e40ea258dc
**Issue:** The UI was creating two web socket connections to receive notification updates causing duplicated values. **Cause:** We were rendering the notification container twice. One for the desktop nav and another for mobile. **Fix:** Only use one notification container for the nav. **Improvements for later:** I think would be better at some point to move the networking and data logic into a provider but it would require testing and some tiny rework. Since the actual fix works well, and it is not complex or difficult, I think it is ok to stay with it until we require to load notifications in more places.