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.

Sign up for GitHub

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

Jump to bottom

non-displayable maps visible after admin logout #1192

Open
huss opened this issue Apr 3, 2024 · 0 comments · May be fixed by #1314
Open

non-displayable maps visible after admin logout #1192

huss opened this issue Apr 3, 2024 · 0 comments · May be fixed by #1314
Labels
p-low-priority t-bug The issue is related to an aspect of the software which is incorrect
Milestone
2.x

Comments

@huss
Copy link
Member

huss commented Apr 3, 2024

Describe the bug

After an admin logs out, the now non-admin user can work with these maps.

To Reproduce

Steps to reproduce the behavior:

  1. login as admin
  2. go to maps admin page and set map to display disabled
  3. graph something on this map (not really needed)
  4. log out and the map can still be used
  5. refresh the page and the map is not visible

Expected behavior

Only appropriately visible maps should be shown.

Additional context

The suspicion is that logout is not properly invalidating the map state.
@huss huss added t-bug The issue is related to an aspect of the software which is incorrect p-low-priority labels Apr 4, 2024
@huss huss added this to the 1.x milestone Apr 4, 2024
@ChrisMart21 ChrisMart21 linked a pull request Aug 13, 2024 that will close this issue
Open
5 tasks
@ChrisMart21 ChrisMart21 linked a pull request Aug 13, 2024 that will close this issue
Maps Update: Cards and Modals for Maps Components #1314
Open
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
p-low-priority t-bug The issue is related to an aspect of the software which is incorrect
Projects
None yet
Milestone
2.x
Development

Successfully merging a pull request may close this issue.

Maps Update: Cards and Modals for Maps Components juanjoseguva/OEDCST489Fork
1 participant
@huss