-
Notifications
You must be signed in to change notification settings - Fork 5
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
[LW-10289] replace Koralabs handleProvider with SDK handleHttpProvider #1101
Conversation
12945ec
to
347e175
Compare
Allure Report
smokeTests: ❌ test report for 60febca0
|
0bfa6bd
to
7571e0c
Compare
09e87e3
to
d42dc71
Compare
[Cardano.NetworkMagics.Preview]: 'https://preview.api.handle.me' | ||
[Cardano.NetworkMagics.Mainnet]: 'https://live-preprod.lw.iog.io/', | ||
[Cardano.NetworkMagics.Preprod]: 'https://dev-preprod.lw.iog.io/', | ||
[Cardano.NetworkMagics.Preview]: 'https://dev-preview.lw.iog.io/' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't we use the .env vars here instead of hardcoded values?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left some "good to have" suggestions.
apps/browser-extension-wallet/src/features/ada-handle/config.ts
Outdated
Show resolved
Hide resolved
75f4fcd
to
2d0ad44
Compare
apps/browser-extension-wallet/src/utils/validators/address-book.ts
Outdated
Show resolved
Hide resolved
apps/browser-extension-wallet/src/utils/validators/address-book.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@VanessaPC LGTM 🚀
I left just a minor suggestion
9c4346b
to
e214c38
Compare
705519f
to
60febca
Compare
Quality Gate passedIssues Measures |
Checklist
Proposed solution
We want to start using
handleHttpProvider
instead of Koralabs, therefore we replace all instances and clean up imports.Testing
Screenshots