Skip to content

Minds Frontend
Minds Frontend
Open
Opened by Ben Hayward@benhayward.ben
Report abuse

[Sprint/TrendyTurtle](feat): Added in error message on socket connection breaking. #2115

Summary

Closes #2115 MR to add an error message in to show the user when their connection to live comments has been lost.

Steps to test

To discuss - may need to make some API endpoints so that we can trigger this on sandboxes.

Estimated Regression Scope

The worst-case scenario is are that something could break in sockets or thread comments. E.g. the error showing at inappropriate times if the error is for some reason called more times than necessary on prod, or maybe the load more of the comments thread.

Request to merge feat/socket-error-comments-2115 into master
The source branch is 6 commits behind the target branch
Open in Web IDE
Pipeline #102021616 failed for 7c1778f5 on feat/socket-error-comments-2115
              Requires 3 more approvals from Devs, Deployers, and QA.
              Ready to be merged automatically. Ask someone with write access to this repository to merge this request

              Closes #2115

              Deletes source branch

              Please register or sign in to reply