Skip to content

chore: Add unified error handling banner in all demo apps#981

Merged
msluszniak merged 4 commits intomainfrom
@ms/add-unified-error-indicator-to-demo-apps
Mar 23, 2026
Merged

chore: Add unified error handling banner in all demo apps#981
msluszniak merged 4 commits intomainfrom
@ms/add-unified-error-indicator-to-demo-apps

Conversation

@msluszniak
Copy link
Member

@msluszniak msluszniak commented Mar 18, 2026

Description

This PR adds unified error banner that shows the error details.

Introduces a breaking change?

  • Yes
  • No

Type of change

  • Bug fix (change which fixes an issue)
  • New feature (change which adds functionality)
  • Documentation update (improves or adds clarity to existing documentation)
  • Other (chores, tests, code style improvements etc.)

Tested on

  • iOS
  • Android

Testing instructions

  • Run demo apps and try to cause errors that propagates from our hooks / models (not some random errors).

Screenshots

Banners should look like this:
image

Related issues

Closes #225

Checklist

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have updated the documentation accordingly
  • My changes generate no new warnings

Additional notes

@msluszniak msluszniak self-assigned this Mar 18, 2026
@msluszniak msluszniak added chore PRs that are chores demo app labels Mar 18, 2026
@barhanc barhanc self-requested a review March 19, 2026 13:41
@NorbertKlockiewicz
Copy link
Contributor

computer-vision:

  • classification/index.tsx
  • ocr_vertical/index.tsx
  • vision_camera/index.tsx

llm:

  • multimodal_llm/index.tsx

Those screens are missing the new error handling

@msluszniak msluszniak force-pushed the @ms/add-unified-error-indicator-to-demo-apps branch from 9b51872 to 51387de Compare March 23, 2026 13:04
@msluszniak msluszniak merged commit 27592da into main Mar 23, 2026
4 checks passed
@msluszniak msluszniak deleted the @ms/add-unified-error-indicator-to-demo-apps branch March 23, 2026 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore PRs that are chores demo app

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add visual error indicator to example apps.

3 participants