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

Missing configuration creates runtime error that's unfriendly for beginners #38

Open
matthewpereira opened this issue Jul 25, 2024 · 0 comments
Labels
good first issue Good for newcomers

Comments

@matthewpereira
Copy link
Contributor

Expected behavior

If a user charges ahead with running the sample application before configuring their Auth0 tenant or environment configuration, the sample application should detect the problem and point them in the right direction to rectify the situation.

Actual behavior

image

Proposal

Adding a try/catch that either triggers an error page component that we define elsewhere, or an error.message that we define inline on the app/page.tsx like this:

image

We can use this error to point users toward the README.md / bootstrapping script, or direct them to manually edit their .env file to ensure that the secret value (or perhaps other values that might be missing) are correct.

Feedback welcome on how we'd like to handle error cases so that it's scalable, and doesn't clutter up too many files.

@matthewpereira matthewpereira added the good first issue Good for newcomers label Jul 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant