bob-collective / ui

BOB is a hybrid L2 that combines the security of Bitcoin with the versatility of Ethereum
0 stars 0 forks source link

feat: refactor bridge and add placeholder tx #278

Closed danielsimao closed 1 week ago

danielsimao commented 2 weeks ago

Closes #

📝 Description

Part of this code was already reviewed by @tomjeatt. I had to close older PR because it was very outdated. In this PR I took the opportunity to clean-up the types around the bridge. Most of the cleaned up code was from the first bridge iterations.

You will see that I replace a Type enum with a more assertive TransactionDirection.

The main change of this PR should be around how we handle the data in the TransactionList along with hooks for getting transactions.

Also removed gateway feature flag

Fixes majors issues in Transaction List loading.

vercel[bot] commented 2 weeks ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
bob-mainnet ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 4, 2024 11:28am
bob-sepolia ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 4, 2024 11:28am
storybook ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 4, 2024 11:28am
changeset-bot[bot] commented 2 weeks ago

⚠️ No Changeset found

Latest commit: 4df6cd448bbca1e8d63e531b94e11a0cc61f6541

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR