Skip to content

Update comments#1324

Merged
victorlin merged 1 commit intomasterfrom
victorlin/update-error-handling
Mar 3, 2026
Merged

Update comments#1324
victorlin merged 1 commit intomasterfrom
victorlin/update-error-handling

Conversation

@victorlin
Copy link
Member

@victorlin victorlin commented Mar 3, 2026

Update the first comment to better explain the purpose of this code block, which only serves very specific routes such as '/charon/invalid'.

The nested comment was also not entirely accurate, as Next.js can respond with 404 for some routes. I lifted it out and rewrote as a proper TODO based on the linked issues.

Checklist

@victorlin victorlin self-assigned this Mar 3, 2026
Update the first comment to better explain the purpose of this code
block, which only serves very specific routes such as '/charon/invalid'.

The nested comment was also not entirely accurate, as Next.js can
respond with 404 for some routes. I lifted it out and rewrote as a
proper TODO based on the linked issues.
@victorlin victorlin force-pushed the victorlin/update-error-handling branch from 24cda24 to 6520f08 Compare March 3, 2026 18:09
@victorlin victorlin merged commit e4edf89 into master Mar 3, 2026
7 checks passed
@victorlin victorlin deleted the victorlin/update-error-handling branch March 3, 2026 18:10
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