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 injective resume transaction #1866

Merged
merged 1 commit into from
Apr 9, 2024

Conversation

yuli-ferna
Copy link
Collaborator

@yuli-ferna yuli-ferna commented Apr 2, 2024

Issue: XLabs/portal-bridge-ui#795

Polkachu's rpc is not bringing the info of many injective transactions, the desired solution would be to use the injective labs library and do the following https://docs.ts.injective.network/querying/querying-api/querying-indexer-explorer#fetch-transaction-using-transaction-hash. But because the package @certusone/wormhole-sdk has old versions of these libraries, it generates conflict, so for now I have here the additional solution which is to use a different rpc (https://injective-rpc.publicnode.com/) while we can solve the problem.

image

Copy link

netlify bot commented Apr 2, 2024

Deploy Preview for wormhole-connect-builder ready!

Name Link
🔨 Latest commit b23aed9
🔍 Latest deploy log https://app.netlify.com/sites/wormhole-connect-builder/deploys/661582217cf9d100080c22e6
😎 Deploy Preview https://deploy-preview-1866--wormhole-connect-builder.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Apr 2, 2024

Deploy Preview for wormhole-connect-mainnet ready!

Name Link
🔨 Latest commit b23aed9
🔍 Latest deploy log https://app.netlify.com/sites/wormhole-connect-mainnet/deploys/66158221febb3900088d0cac
😎 Deploy Preview https://deploy-preview-1866--wormhole-connect-mainnet.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Apr 2, 2024

Deploy Preview for wormhole-connect ready!

Name Link
🔨 Latest commit b23aed9
🔍 Latest deploy log https://app.netlify.com/sites/wormhole-connect/deploys/66158221febb3900088d0caa
😎 Deploy Preview https://deploy-preview-1866--wormhole-connect.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

artursapek
artursapek previously approved these changes Apr 2, 2024
@yuli-ferna yuli-ferna marked this pull request as ready for review April 2, 2024 19:28
kev1n-peters
kev1n-peters previously approved these changes Apr 9, 2024
@yuli-ferna yuli-ferna dismissed stale reviews from kev1n-peters and artursapek via bc04898 April 9, 2024 17:57
@yuli-ferna yuli-ferna force-pushed the bugfix/injective-resume-tx branch from bc04898 to b23aed9 Compare April 9, 2024 18:00
@kev1n-peters kev1n-peters merged commit 1d5c15c into development Apr 9, 2024
19 checks passed
@kev1n-peters kev1n-peters deleted the bugfix/injective-resume-tx branch April 9, 2024 18:14
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.

3 participants