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

chore: 597 update error boundary message #2973

Merged
merged 1 commit into from
Mar 12, 2025

Conversation

shon-button
Copy link
Contributor

@shon-button shon-button commented Mar 8, 2025

Addresses: 597

🚀 Impact:

  • Updated ErrorBoundary message to be more user friendly instead of Next.js generic masked error details message
  • Included the reference id of the Sentry log in the error message details
  • Removed (production masked) error details in utils fetch functions

🔬 Local Testing:

Tests Set-Up:

  1. Start the API server:

    cd bc_obps
    make reset_db
    make run
  2. Start the app development server:

    cd bciers && yarn dev-all
  3. Navigate to http://localhost:3000

  4. Click the "Log in with Business BCeID" button


Test: Error Message``

Steps:

  1. Navigate to reporting url using an incorrect report version id, ex:

Expected Result

image

@shon-button shon-button changed the title chore: update error boundary message chore: 597 update error boundary message Mar 8, 2025
@shon-button shon-button force-pushed the chore/597-friendly-error-message branch from a032cde to 5fed605 Compare March 9, 2025 18:57
@shon-button shon-button force-pushed the chore/597-friendly-error-message branch from 5fed605 to 106fcfb Compare March 12, 2025 11:51
@shon-button shon-button merged commit e553f95 into develop Mar 12, 2025
23 checks passed
@shon-button shon-button deleted the chore/597-friendly-error-message branch March 12, 2025 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants