Skip to content

Cache user locally#1851

Merged
gaspergrom merged 4 commits intomainfrom
bugfix/cache-user
Nov 21, 2023
Merged

Cache user locally#1851
gaspergrom merged 4 commits intomainfrom
bugfix/cache-user

Conversation

@gaspergrom
Copy link
Copy Markdown
Contributor

@gaspergrom gaspergrom commented Nov 17, 2023

Changes proposed ✍️

What

🤖[deprecated] Generated by Copilot at bdbc23e

The pull request enhances the auth module by using local storage to cache and retrieve user data. This improves the speed and reliability of the authentication process and reduces the server load.

🤖[deprecated] Generated by Copilot at bdbc23e

We are the rebels of the web
We don't need servers to fetch our data
We store it locally in our auth module
We clear it all when we sign out of hell

Why

How

🤖[deprecated] Generated by Copilot at bdbc23e

  • Store and retrieve user data in local storage to improve performance and avoid unnecessary requests (link, link)
  • Remove automatic signout and mutation on initialization error to prevent user inconvenience (link)
  • Clear user data from local storage on signout to prevent data leakage (link)

Checklist ✅

  • Label appropriately with Feature, Improvement, or Bug.
  • Add screenshots to the PR description for relevant FE changes
  • New backend functionality has been unit-tested.
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.

@gaspergrom gaspergrom self-assigned this Nov 17, 2023
@gaspergrom gaspergrom added the Improvement Created by Linear-GitHub Sync label Nov 17, 2023
@gaspergrom gaspergrom merged commit 11a0597 into main Nov 21, 2023
@gaspergrom gaspergrom deleted the bugfix/cache-user branch November 21, 2023 09:27
gaspergrom added a commit that referenced this pull request Nov 21, 2023
gaspergrom added a commit that referenced this pull request Nov 21, 2023
Co-authored-by: Joana Maia <joana@crowd.dev>
(cherry picked from commit 11a0597)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Improvement Created by Linear-GitHub Sync

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants