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

Avoid removing snapshots when switching between users #834

Draft

sejas wants to merge 2 commits into trunk from update/keep-saved-snapshots-between-users

Member

sejas commented Jan 23, 2025 •

edited

Loading

Warning

WIP, this draft PR just started and is incomplete
The Proposed Changes are the plan not a reality

Related issues

Fixes https://github.com/Automattic/dotcom-forge/issues/7516

Proposed Changes

Let's keep all the snapshots in appData
Let's add a new field userId
Let's filter in the UI the snapshots that belong to the current user
Let's figure it out what snapshots should be "removed" from appData and when.
Updated tests

Testing Instructions

Run npm start
Authenticate with user A
Create a demo site
Log out
Authenticate with user B
Observe you don't see any demo site
Log out
Authenticate with user A again
Observe you see the demo site you've created in step 3

Pre-merge Checklist

Have you checked for TypeScript, React or other console errors?


          Avoid clearing snapshots when empty array is received from JN list en…

2dadd78

…dpoint

sejas self-assigned this


          Merge branch 'trunk' of github.com:Automattic/studio into update/keep…

e92933e

…-saved-snapshots-between-users

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

Labels

None yet