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

fix: fix issue with not possible to buy tokens #1098

Merged

Conversation

ost-ptk
Copy link
Member

@ost-ptk ost-ptk commented Nov 25, 2024

Make sure to fill in all the below sections.

Description

  • Refactor the error handling logic by introducing a handleError callback to streamline error management across functions.
  • Added a new state variable providerUrl and adjusted the logic to set this URL before opening it, ensuring consistency and reducing redundancy.

Linked tickets

WALLET-451

Checklist

  • Make sure this PR title follows semantic release conventions: https://semantic-release.gitbook.io/semantic-release/#commit-message-format

  • If the PR adds any new text to the UI, make sure they are localized

  • Include a screenshot or recording if implementing significant UI or user flow change

  • When this PR affects architecture changes wait for review from Dmytro before merging

@ost-ptk ost-ptk requested a review from Comp0te November 25, 2024 15:37
@Comp0te Comp0te merged commit 3015b33 into develop Dec 9, 2024
5 checks passed
@Comp0te Comp0te deleted the WALLET-451-cw-firefox-it-is-not-possible-to-buy-tokens branch December 9, 2024 21:32
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