Skip to content

[PB-6505] Use set user#2024

Merged
TamaraFinogina merged 9 commits into
masterfrom
use_setUser
Jul 13, 2026
Merged

[PB-6505] Use set user#2024
TamaraFinogina merged 9 commits into
masterfrom
use_setUser

Conversation

@TamaraFinogina

@TamaraFinogina TamaraFinogina commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Description

Removes the unused email field and switches to the setUser function instead of setting it directly in local storage.

Related Issues

Relates to PB-6505

Checklist

  • Changes have been tested locally.
  • Unit tests have been written or updated as necessary.
  • The code adheres to the repository's coding standards.
  • Relevant documentation has been added or updated.
  • No new warnings or errors have been introduced.
  • SonarCloud issues have been reviewed and addressed.
  • QA Passed

Testing Process

unit tests

Additional Notes

setUser/getUser will be needed for adding encryption

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 6, 2026

Copy link
Copy Markdown

Deploying drive-web with  Cloudflare Pages  Cloudflare Pages

Latest commit: 975f688
Status:⚡️  Build in progress...

View logs

@TamaraFinogina TamaraFinogina self-assigned this Jul 6, 2026
@TamaraFinogina TamaraFinogina marked this pull request as ready for review July 6, 2026 16:56
Comment thread src/services/local-storage.service.ts Outdated
@TamaraFinogina TamaraFinogina requested a review from larryrider July 7, 2026 13:08
@sonarqubecloud

Copy link
Copy Markdown

@TamaraFinogina TamaraFinogina merged commit 97266bf into master Jul 13, 2026
10 of 11 checks passed
@TamaraFinogina TamaraFinogina deleted the use_setUser branch July 13, 2026 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants