Skip to content

feat(checkout): display error on ID.me redirect with idme_error param#715

Merged
dylandepass merged 1 commit into
staginguatfrom
issue-509-checkout-conditionally-display-err
Jul 15, 2026
Merged

feat(checkout): display error on ID.me redirect with idme_error param#715
dylandepass merged 1 commit into
staginguatfrom
issue-509-checkout-conditionally-display-err

Conversation

@dylandepass

Copy link
Copy Markdown
Contributor

Fixes #509

After ID.me authorization, the redirect back to checkout may carry an idme_error query param (e.g. token_exchange_failed) indicating verification failed. Read the param on page load, clean it from the URL, and show the localized error in the existing checkout error banner so users get clear feedback instead of a silent failure.

Test URLs:

After ID.me authorization, the redirect back to checkout may carry an idme_error query param (e.g. token_exchange_failed) indicating verification failed. Read the param on page load, clean it from the URL, and show the localized error in the existing checkout error banner so users get clear feedback instead of a silent failure.
@aem-code-sync

aem-code-sync Bot commented Jul 15, 2026

Copy link
Copy Markdown

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
In case there are problems, just click a checkbox below to rerun the respective action.

  • Re-run all PSI checks
  • Re-run failed PSI checks
  • Re-sync branch
Commits

@aem-code-sync

aem-code-sync Bot commented Jul 15, 2026

Copy link
Copy Markdown
Page Scores Audits Google
📱 / PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
🖥️ / PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

@dylandepass
dylandepass requested a review from maxakuru July 15, 2026 19:53
@dylandepass
dylandepass merged commit a6dc778 into staginguat Jul 15, 2026
3 checks passed
@dylandepass
dylandepass deleted the issue-509-checkout-conditionally-display-err branch July 15, 2026 20:51
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