Skip to content
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

React Asset: Improved Error messaging #376

Closed
hborawski opened this issue Jun 5, 2024 · 0 comments
Closed

React Asset: Improved Error messaging #376

hborawski opened this issue Jun 5, 2024 · 0 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@hborawski
Copy link
Contributor

Is your feature request related to a problem? Please describe.

The ReactAsset component handles fetching the implementation from the asset registry, and throwing errors if an asset is not found. These errors indicate the direct cause, but could do a better job guiding the user to the root cause.

We should enhance this messaging to help the user self resolve the issue when possible.

Describe the solution you'd like

  • Add additional metadata to the existing error messages to include what assets and registrations the current configuration supports.
  • Add some react-like errors that give suggestions when there are multiple potential causes
    • Mismatched React Context could show no assets registered, but could be caused by no plugins being added to the Player instance.

Additional context

@hborawski hborawski added enhancement New feature or request good first issue Good for newcomers labels Jun 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants