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

Trend page is not refresh friendly #177

Closed
eidekrist opened this issue Jun 2, 2020 · 0 comments · Fixed by #179
Closed

Trend page is not refresh friendly #177

eidekrist opened this issue Jun 2, 2020 · 0 comments · Fixed by #179
Assignees
Labels
bug Something isn't working

Comments

@eidekrist
Copy link
Member

eidekrist commented Jun 2, 2020

Copied from #143:

Scenario 2:

Setup

  • Run cse-server-go (start it again), load house demo, press play

Action:

  • While running, have one of the trends page open. Press refresh

Result:

  • Models don't disappear (ok)
  • Trend freezes (not ok)
  • A lot of data is dumped to the terminal (not ok)

The client sends which plot is active as the first command after refresh, but does not have enough data to know what the actual id of the plot is, only the index. A possible solution could be to use the plot id as route parameter instead of index.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant