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

Cached parts of the interface by switching between accounts not updating #6549

Closed
elizavetaRa opened this issue Mar 8, 2022 · 2 comments · Fixed by #6694
Closed

Cached parts of the interface by switching between accounts not updating #6549

elizavetaRa opened this issue Mar 8, 2022 · 2 comments · Fixed by #6694

Comments

@elizavetaRa
Copy link
Member

Steps to reproduce

  1. In CERNBox login with a normal account
  2. Logout and login with the lightweight account, that gets another list of navItems and doesn't have quota

Implementation of different lists of navItems here: https://github.com/cernbox/web/blob/0dfa4717578cedfaa0cf976e027b20c212f76b6f/packages/web-app-files/src/index.js#L169-L171

Expected behaviour

The navItems and the quota are changed corresponding to the account

Actual behaviour

The list of navItems for the lightweight accounts is mixed with cached navItems from previous account. Quota is shown where it doesn't exist. By opening the app in new tab the problem disappears.

Untitled_ Mar 8, 2022 11_49 AM

@elizavetaRa elizavetaRa changed the title Cached parts of the interface by switching between accounts Cached parts of the interface by switching between accounts not updating Mar 8, 2022
@elizavetaRa
Copy link
Member Author

elizavetaRa commented Mar 23, 2022

@kulmann I found the issue with quota: if the backend returns no quota (our case with lightweight accounts), the quota in store is not reset from the previous logged in user inside one browser tab.

Similar happens to navItems, by change of the user who has other navItems, those get added to navItems from previous user.

@diocas
Copy link
Contributor

diocas commented Mar 24, 2022

@elizavetaRa I guess this can be reproduced in master if you - for example - add a new menu entry on the left sidebar?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants