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

chore: upgrade unified bridge SDK #40

Merged
merged 27 commits into from
Oct 24, 2024
Merged

chore: upgrade unified bridge SDK #40

merged 27 commits into from
Oct 24, 2024

Conversation

meeh0w
Copy link
Member

@meeh0w meeh0w commented Sep 10, 2024

Description

TODOs

  • Release @avalabs/bridge-unified to a stable channel

Changes

  • Upgrades @avalabs/bridge-unified dependency
    • The new version includes some breaking changes. This PR updates our code to make it compatible.
  • Refactors how we "recognise" CCTP (and the upcoming ICTT) bridge transactions - previously we would look at UnifiedBridge config to fetch the addresses, with this PR the updated SDK exposes the analyzeTx() method that we can use.
  • Implements a schema migration for any pending bridge transfers (the BridgeTransfer object changed its shape, some properties changed names and/or places)

Testing

  • Test bridging. It should work exactly the same as it used to.
  • Run sanity checks on other parts of the extension
  • Bonus points for simulating the extension update (initiate bridging from main, switch to this branch and verify that your transaction is still tracked properly).

Screenshots:

Starts bridging via CCTP on main, simulates the extension update, then starts another bridge with the "updated" extension:

CCTP.after.update.-.compressed.mov

Checklist for the author

  • I've covered new/modified business logic with Jest test cases.
  • I've tested the changes myself before sending it to code review and QA.

gergelylovas
gergelylovas previously approved these changes Oct 3, 2024
src/contexts/UnifiedBridgeProvider.tsx Outdated Show resolved Hide resolved
gergelylovas
gergelylovas previously approved these changes Oct 22, 2024
Copy link
Contributor

@gergelylovas gergelylovas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏

package.json Outdated
@@ -23,10 +23,10 @@
"sentry": "node sentryscript.js"
},
"dependencies": {
"@avalabs/avalanche-module": "0.11.2",
"@avalabs/avalanche-module": "^0.11.2",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you please pin the dependencies?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😱 done!

@meeh0w meeh0w merged commit 68a849c into main Oct 24, 2024
5 checks passed
@meeh0w meeh0w deleted the feat/ictt branch October 24, 2024 09:59
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