parse-dashboard icon indicating copy to clipboard operation
parse-dashboard copied to clipboard

Using browser navigation clears data browser

Open mtrezza opened this issue 3 years ago • 4 comments

New Issue Checklist

Issue Description

Navigating using the web browser's "back" and "forward" buttons doesn't shows empty table in the data browser. The data browser briefly shows a loading screen, then the rows in the data browser, then clears the data browser.

It also seems that the dashboard is at times completely reloading itself. This causes the whole dashboard to be reloaded from the server, which seems unnecessary.

It seems, this bug has been introduced with https://github.com/parse-community/parse-dashboard/pull/2273.

Steps to reproduce

Navigate by clicking on a pointer in a row in the data browser, then again on a pointer, thus drilling down in the data hierarchy. Then navigate back and forth using the web browser buttons.

Actual Outcome

The data browser is empty.

Expected Outcome

The data browser should show the correct data.

Environment

Dashboard

  • Parse Dashboard version: 5.0.0-alpha.3

mtrezza avatar Oct 04 '22 17:10 mtrezza

Thanks for opening this issue!

  • 🚀 You can help us to fix this issue faster by opening a pull request with a failing test. See our Contribution Guide for how to make a pull request, or read our New Contributor's Guide if this is your first time contributing.

@magdaraa In case you are interested in taking a look at this.

mtrezza avatar Oct 04 '22 17:10 mtrezza

Hey, thank you for your suggestion. I have just started: https://github.com/parse-community/parse-dashboard/issues/2239 I'm happy to look into that next.

Magdaraa avatar Oct 04 '22 21:10 Magdaraa

@Magdaraa it seems that the issue is not caused by #2273 but by https://github.com/parse-community/parse-dashboard/pull/2275, but we don't know yet.

mtrezza avatar Oct 12 '22 20:10 mtrezza

This bug currently blocks the beta release

mtrezza avatar Oct 15 '22 22:10 mtrezza

Closing via #2317

mtrezza avatar Oct 17 '22 16:10 mtrezza