Skip to content

fix: BROS-136: Don't allow to create regions in View All #7907

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
Jul 7, 2025

Conversation

hlomzik
Copy link
Collaborator

@hlomzik hlomzik commented Jul 3, 2025

Add checks for annotation.isReadonly() wherever is more appropriate and precise. Regions creation is fixed for these tags:

  • Audio
  • Paragraphs
  • TimeSeries
  • Video Timeline

Also video region interpolation controls are disabled in readonly mode.

  • linting
  • remove unused Audio/model#createRegion()

@hlomzik hlomzik requested review from a team, Gondragos and nick-skriabin as code owners July 3, 2025 18:51
Copy link

netlify bot commented Jul 3, 2025

Deploy Preview for label-studio-docs-new-theme ready!

Name Link
🔨 Latest commit c1ac84f
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-docs-new-theme/deploys/686bc4262913460008480baa
😎 Deploy Preview https://deploy-preview-7907--label-studio-docs-new-theme.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

netlify bot commented Jul 3, 2025

Deploy Preview for label-studio-playground ready!

Name Link
🔨 Latest commit c1ac84f
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-playground/deploys/686bc426436e00000887de80
😎 Deploy Preview https://deploy-preview-7907--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

netlify bot commented Jul 3, 2025

Deploy Preview for heartex-docs ready!

Name Link
🔨 Latest commit c1ac84f
🔍 Latest deploy log https://app.netlify.com/projects/heartex-docs/deploys/686bc4265ed8490008c26871
😎 Deploy Preview https://deploy-preview-7907--heartex-docs.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 Jul 3, 2025
Copy link

netlify bot commented Jul 3, 2025

Deploy Preview for label-studio-storybook ready!

Name Link
🔨 Latest commit c1ac84f
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-storybook/deploys/686bc426b9ea170008509d26
😎 Deploy Preview https://deploy-preview-7907--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.

Copy link

codecov bot commented Jul 3, 2025

Codecov Report

Attention: Patch coverage is 70.37037% with 8 lines in your changes missing coverage. Please review.

Project coverage is 65.42%. Comparing base (5367e4e) to head (c1ac84f).
Report is 3 commits behind head on develop.

Files with missing lines Patch % Lines
...src/components/TimeSeries/TimeSeriesVisualizer.jsx 0.00% 3 Missing ⚠️
...libs/editor/src/tags/object/TimeSeries/Channel.jsx 60.00% 2 Missing ⚠️
...b/libs/editor/src/components/Timeline/Timeline.tsx 0.00% 1 Missing ⚠️
web/libs/editor/src/tags/object/Audio/model.js 88.88% 1 Missing ⚠️
...ditor/src/tags/object/Paragraphs/HtxParagraphs.jsx 50.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #7907      +/-   ##
===========================================
- Coverage    69.89%   65.42%   -4.48%     
===========================================
  Files          674      465     -209     
  Lines        48966    32224   -16742     
  Branches      8403     8403              
===========================================
- Hits         34227    21084   -13143     
+ Misses       14739    11140    -3599     
Flag Coverage Δ
pytests ?

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.

hlomzik added 2 commits July 4, 2025 03:39
Also don't allow to change interpolation of VideoRectangle regions
@robot-ci-heartex robot-ci-heartex merged commit a1f16d7 into develop Jul 7, 2025
84 of 89 checks passed
@robot-ci-heartex robot-ci-heartex deleted the fb-bros-136/readonly-view-all branch July 7, 2025 15:06
hlomzik added a commit that referenced this pull request Jul 8, 2025
- BROS-136: Don't allow to create regions in View All (#7907)
- BROS-156: Always show Fix+Accept when we have draft (#7897)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants