Skip to content

feat: improve error reporting for invalid magic links#210

Merged
ottokruse merged 1 commit intoaws-samples:mainfrom
aashirjaved:improve-invalid-magic-link-error-reporting
Nov 11, 2024
Merged

feat: improve error reporting for invalid magic links#210
ottokruse merged 1 commit intoaws-samples:mainfrom
aashirjaved:improve-invalid-magic-link-error-reporting

Conversation

@aashir-motorway
Copy link
Contributor

@aashir-motorway aashir-motorway commented Nov 7, 2024

Overview

This PR enhances error visibility for invalid magic link handling in the UI. Currently, an invalid magic link would throw a generic error message Ignoring invalid fragment identifier, which provided minimal context and made debugging challenging.

Screenshot 2024-11-07 at 12 32 18

It took us sometime to identify the issue and this PR introduces more descriptive error handling to help quickly understand the root cause with invalid magic links.

Results:

Screenshot 2024-11-07 at 12 53 53

@aashir-motorway aashir-motorway force-pushed the improve-invalid-magic-link-error-reporting branch from a164c2e to 49ad1d1 Compare November 7, 2024 13:07
@aashir-motorway aashir-motorway force-pushed the improve-invalid-magic-link-error-reporting branch from 6b85b03 to cb9a979 Compare November 9, 2024 08:28
feat: improve error reporting for invalid magic links

feat: improve error reporting for invalid magic links

fix: unkown template literal error

fix: unkown template literal error
@aashir-motorway aashir-motorway force-pushed the improve-invalid-magic-link-error-reporting branch from cb9a979 to 91fc996 Compare November 9, 2024 08:29
@ottokruse ottokruse merged commit 834f5bd into aws-samples:main Nov 11, 2024
@ottokruse
Copy link
Contributor

Thank you!

@ottokruse
Copy link
Contributor

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.

3 participants