Skip to content

fix(payments-next): Add error handling to Free trial fetching#20382

Open
elizabeth-ilina wants to merge 1 commit intomainfrom
PAY-3634-add-error-handling-to-free-trial-fetching
Open

fix(payments-next): Add error handling to Free trial fetching#20382
elizabeth-ilina wants to merge 1 commit intomainfrom
PAY-3634-add-error-handling-to-free-trial-fetching

Conversation

@elizabeth-ilina
Copy link
Copy Markdown
Contributor

@elizabeth-ilina elizabeth-ilina commented Apr 15, 2026

Because

  • Right now, if an error is thrown when fetching free trial info, the checkout page crashes.

This pull request

  • Wraps the fetch of free trial data in a try catch

Issue that this pull request solves

Closes #PAY-3634

Checklist

Put an x in the boxes that apply

  • My commit is GPG signed.
  • If applicable, I have modified or added tests which pass locally.
  • I have added necessary documentation (if appropriate).
  • I have verified that my changes render correctly in RTL (if appropriate).
  • I have manually reviewed all AI generated code.

How to review (Optional)

  • Key files/areas to focus on:
  • Suggested review order:
  • Risky or complex parts:

Screenshots (Optional)

Please attach the screenshots of the changes made in case of change in user interface.

Other information (Optional)

Any other information that is important to this pull request.

@elizabeth-ilina elizabeth-ilina marked this pull request as ready for review April 16, 2026 13:57
@elizabeth-ilina elizabeth-ilina requested a review from a team as a code owner April 16, 2026 13:57
Because:

* Right now, if an error is thrown when fetching free trial info, the checkout page crashes.

This commit:

* Wraps the fetch of free trial data in a try catch

Closes #[PAY-3634](https://mozilla-hub.atlassian.net/browse/PAY-3634)
@elizabeth-ilina elizabeth-ilina force-pushed the PAY-3634-add-error-handling-to-free-trial-fetching branch from 5ea9a8b to 6650470 Compare April 16, 2026 14:37
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.

1 participant