Skip to content

Conversation

@bmartel
Copy link
Contributor

@bmartel bmartel commented Apr 29, 2025

This pull request modifies the error handling logic in the ApiProvider component to allow for more flexible inline error handling. The key change is the separation of the setError logic from the global error display logic.

Error Handling Improvements:

  • web/apps/labelstudio/src/providers/ApiProvider.tsx: Introduced a conditional block to handle inline errors separately by calling setError before checking for global error display conditions. This ensures that inline error handling can occur independently of the global error logic.

@github-actions github-actions bot added the fix label Apr 29, 2025
@netlify
Copy link

netlify bot commented Apr 29, 2025

Deploy Preview for label-studio-storybook ready!

Name Link
🔨 Latest commit aedb8de
🔍 Latest deploy log https://app.netlify.com/sites/label-studio-storybook/deploys/68126b36eb55ea00094b3238
😎 Deploy Preview https://deploy-preview-7442--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 site configuration.

@netlify
Copy link

netlify bot commented Apr 29, 2025

Deploy Preview for heartex-docs ready!

Name Link
🔨 Latest commit aedb8de
🔍 Latest deploy log https://app.netlify.com/sites/heartex-docs/deploys/68126b3672f861000861c35d
😎 Deploy Preview https://deploy-preview-7442--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 site configuration.

@netlify
Copy link

netlify bot commented Apr 29, 2025

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

Name Link
🔨 Latest commit aedb8de
🔍 Latest deploy log https://app.netlify.com/sites/label-studio-docs-new-theme/deploys/68126b36906b910008777a9f
😎 Deploy Preview https://deploy-preview-7442--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 site configuration.

@bmartel bmartel requested a review from yyassi-heartex April 29, 2025 22:04
@AndrejOros
Copy link
Contributor

AndrejOros commented Apr 30, 2025

/fm sync

Workflow run

@bmartel bmartel merged commit 1aab1d2 into develop Apr 30, 2025
40 checks passed
@robot-ci-heartex robot-ci-heartex deleted the fb-optic-1982 branch April 30, 2025 21:53
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