Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clients get out of sync from centralised datastore #679

Open
joepavitt opened this issue Mar 14, 2024 · 3 comments
Open

Clients get out of sync from centralised datastore #679

joepavitt opened this issue Mar 14, 2024 · 3 comments
Labels
bug Something isn't working priority:high High Priority size:M - 3 Sizing estimation point

Comments

@joepavitt
Copy link
Collaborator

Current Behavior

If you have two separate Dashboard clients open and connected to a centralised Node-RED instance with Dashboard 2.0, the server has a single source of truth.

If one client updates that data, e.g. through a toggle switch, the centralised store of data is updated. However, other clients are not aware of this change until they refresh, which is not the desired behaviour. Any clients connected should update if a client has been the driver of change (i.e. user clicks a toggle).

This already works for all instance where a node is injected with a new value.

Expected Behavior

All clients should reflect the server-side datastore at all times, unless using multi-user authentication/Dashboards architecture

Steps To Reproduce

  1. Add a toggle switch
  2. Open two browsers connecting to the Dashboard
  3. Toggle the switch on one of the clients
  4. Note that the other client doesn't automically update

Environment

  • Dashboard version: 1.4.0

Have you provided an initial effort estimate for this issue?

I have provided an initial effort estimate

@joepavitt joepavitt added size:M - 3 Sizing estimation point bug Something isn't working needs-triage Needs looking at to decide what to do labels Mar 14, 2024
@joepavitt
Copy link
Collaborator Author

This will require a little re-think of our underlying architecture, as this should just be working out of the box, but the events flow doesn't allow for it currently.

@joepavitt joepavitt added priority:high High Priority and removed needs-triage Needs looking at to decide what to do labels Mar 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority:high High Priority size:M - 3 Sizing estimation point
Projects
Status: Up Next
Development

No branches or pull requests

2 participants