m2ms / fragalysis-frontend

The React, Redux frontend built by webpack
Other
1 stars 2 forks source link

RHS CSets incorrectly persist when switching targets #1195

Open mwinokan opened 1 year ago

mwinokan commented 1 year ago

After changing targets, the compound sets listed in the RHS will show those from the previous target. See images below.

mwinokan commented 1 year ago

RHS for D68EV3C:

Screenshot 2023-11-10 at 08 54 51

RHS after switching to A71EV2A:

Screenshot 2023-11-10 at 08 55 03

A71EV2A has had no RHS uploads so it should look like this:

Screenshot 2023-11-10 at 08 55 20

boriskovar-m2ms commented 1 year ago

What are the steps to reproduce? For me it works A71EV2A has three computed sets from Matteo now though. I go back to landing page and pick another target and it works. This is one of the many reasons why there is that reload when you get back to landing page because there was many garbage not reset in the redux store so it became unmaintainable because the sheer amount of data/state stored in the store caused every now and then something to slip through the cracks so that's why it now "hard resets" when you navigate back to the landing page so in theory this should not be possible but it clearly is.

mwinokan commented 1 year ago

I didn't do anything special. I was just able to reproduce the error by:

boriskovar-m2ms commented 1 year ago

Oh I see. "Home" button doesn't trigger this hard reset of the state (basically reloading the page). Forgot that this button even exists... or the whole hamburger menu. Will be easy to bring it in line with behavior when you click on the target in the header