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.

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

fix(core): prevent data loss (hot-fix) #5798

Merged
merged 1 commit into from
Feb 5, 2024
Merged

Conversation

EYHN
Copy link
Member

@EYHN EYHN commented Feb 4, 2024

No description provided.

@github-actions github-actions bot added mod:infra Environment related issues and discussions app:core labels Feb 4, 2024
Copy link

nx-cloud bot commented Feb 4, 2024

@EYHN EYHN changed the title fix(core): avoid data loss (hot-fix) fix(core): prevent data loss (hot-fix) Feb 4, 2024
@Brooooooklyn Brooooooklyn merged commit d2acd03 into beta Feb 5, 2024
31 checks passed
@Brooooooklyn Brooooooklyn deleted the eyhn/fix/data-lose branch February 5, 2024 02:54
@@ -67,12 +73,34 @@ export const Component = (): ReactElement => {
localStorage.setItem('last_workspace_id', workspace.id);
}, [setCurrentWorkspace, meta, workspaceManager, workspace]);

const [workspaceIsLoading, setWorkspaceIsLoading] = useState(true);

// hotfix: avoid doing operation, before workspace is loaded
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe combine this with doc.whenSynced?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app:core mod:infra Environment related issues and discussions
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

3 participants