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

Use LocalStorage instead of cookies for browser state #103

Closed
OhKai opened this issue Sep 28, 2023 · 0 comments · Fixed by #135
Closed

Use LocalStorage instead of cookies for browser state #103

OhKai opened this issue Sep 28, 2023 · 0 comments · Fixed by #135
Assignees
@MaxiLein

Comments

@OhKai
Copy link
Contributor

OhKai commented Sep 28, 2023

Is your feature request related to a problem? Please describe.
Currently, the user preferences which can be considered "browser state" (they should be stored on a per-browser basis instead of per-user) are stored in a cookie in getPreferences().

Describe the solution you'd like
Since cookies are sent to the server on every request, I think it would be better suited for LocalStorage.
@OhKai OhKai moved this to Todos: Fine-Grained in PROCEED MS New Sep 28, 2023
@MaxiLein MaxiLein linked a pull request Oct 19, 2023 that will close this issue
User preferences localstorage #123
Closed
This was referenced Oct 19, 2023
Closed
Merged
@github-project-automation github-project-automation bot moved this from Todos: Fine-Grained to Done in PROCEED MS New Nov 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MaxiLein MaxiLein

Labels
None yet
Projects
PROCEED MS New
Status: Done
Milestone
No milestone
2 participants
@OhKai @MaxiLein