Skip to content

fix: LEAP-649: Saving draft on task switching #7212

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

Merged
merged 7 commits into from
Mar 20, 2025

Conversation

Gondragos
Copy link
Collaborator

This commit ensures the task ID remains consistent throughout the onSubmitDraft process, even while waiting for asynchronous operations. Previously, the task ID could change during submission, leading to potential issues. This change prevents that.

Preserve the task ID at the start of onSubmitDraft to ensure it's consistent while awaiting async operations. This prevents potential issues caused by task ID updates mid-submission.
@github-actions github-actions bot added the fix label Mar 12, 2025
Copy link

netlify bot commented Mar 12, 2025

Deploy Preview for label-studio-docs-new-theme canceled.

Name Link
🔨 Latest commit c5bff6a
🔍 Latest deploy log https://app.netlify.com/sites/label-studio-docs-new-theme/deploys/67dbce80d397da0008bfa6f2

Copy link

netlify bot commented Mar 12, 2025

Deploy Preview for heartex-docs canceled.

Name Link
🔨 Latest commit c5bff6a
🔍 Latest deploy log https://app.netlify.com/sites/heartex-docs/deploys/67dbce80b946bd0008ce3d8e

@Gondragos Gondragos requested a review from hlomzik March 12, 2025 10:35
@Gondragos
Copy link
Collaborator Author

Gondragos commented Mar 18, 2025

/fm sync

Workflow run

@AndrejOros
Copy link
Contributor

AndrejOros commented Mar 19, 2025

/fm sync

Workflow run

@MihajloHoma
Copy link
Contributor

MihajloHoma commented Mar 20, 2025

/fm sync

Workflow run

@Gondragos Gondragos merged commit a01b255 into develop Mar 20, 2025
34 checks passed
@Gondragos Gondragos deleted the fb-LEAP-649/wrong-task-draft branch March 20, 2025 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants