Skip to content

fix: OPTIC-1982: Allow any errors to be displayed inline for external connection testing #7442

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
Apr 30, 2025

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
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.

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.

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.

4 participants