Skip to content

fix: UTC-22: Validation error when updating task data via API #7644

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 4 commits into from
May 30, 2025

Conversation

mcanu
Copy link
Contributor

@mcanu mcanu commented May 28, 2025

There was an issue where an existing project with tasks, changed labeling config. When updating the data of the tasks through the API it failed because the current task data was checked instead of the new data. This PR makes the TaskValidator validates the new data when updated instead of the current one.

Copy link

netlify bot commented May 28, 2025

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

Name Link
🔨 Latest commit 9ff40a9
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-docs-new-theme/deploys/6839d778c53f1c0008faac82

Copy link

netlify bot commented May 28, 2025

Deploy Preview for heartex-docs canceled.

Name Link
🔨 Latest commit 9ff40a9
🔍 Latest deploy log https://app.netlify.com/projects/heartex-docs/deploys/6839d7781a86fd0008506d50

Copy link

netlify bot commented May 28, 2025

Deploy Preview for label-studio-storybook ready!

Name Link
🔨 Latest commit 9ff40a9
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-storybook/deploys/6839d77a29ab3100083589cd
😎 Deploy Preview https://deploy-preview-7644--label-studio-storybook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions bot added the fix label May 28, 2025
Copy link

netlify bot commented May 28, 2025

Deploy Preview for label-studio-playground ready!

Name Link
🔨 Latest commit 9ff40a9
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-playground/deploys/6839d778b01da200091ffa7d
😎 Deploy Preview https://deploy-preview-7644--label-studio-playground.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

codecov bot commented May 28, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.52%. Comparing base (93b6d53) to head (9ff40a9).
Report is 3 commits behind head on develop.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #7644      +/-   ##
===========================================
+ Coverage    78.50%   78.52%   +0.01%     
===========================================
  Files          197      198       +1     
  Lines        16010    16034      +24     
===========================================
+ Hits         12568    12590      +22     
- Misses        3442     3444       +2     
Flag Coverage Δ
pytests 78.52% <100.00%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mcanu mcanu requested a review from hakan458 May 29, 2025 15:36
@mcanu
Copy link
Contributor Author

mcanu commented May 30, 2025

/fm sync

Workflow run

@robot-ci-heartex robot-ci-heartex merged commit 0296399 into develop May 30, 2025
52 checks passed
@robot-ci-heartex robot-ci-heartex deleted the fb-utc-22 branch May 30, 2025 16:40
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.

3 participants