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.

Sign up for GitHub

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

Jump to bottom

feat(bridge-ui-v2): review step #14940

Merged
merged 25 commits into from
Oct 12, 2023
Merged

feat(bridge-ui-v2): review step #14940

merged 25 commits into from
Oct 12, 2023

Conversation

KorbinianK
Copy link
Contributor

@KorbinianK KorbinianK commented Oct 11, 2023 •
edited
Loading

NOT FINAL DESIGN

Second step of the NFT bridge, displaying the selected NFT info for confirmation before bridging

  • Display metdata
  • Added dynamic fetching of NFT metadata
  • Try parsing for image
  • Retry if we hit rate limits or a gateway is down (not dynamic yet)

Also further refinement of the first step (i18n etc).

KorbinianK and others added 19 commits October 2, 2023 16:40
@KorbinianK
Squashed commit of the following:
9268817 
commit 350e935
Author: Kenk <[email protected]>
Date:   Mon Oct 2 20:27:24 2023 +0700

    feat(website): add AIT Protocol to ecosystem (#14868)

commit 70e3034
Author: Vitalik <[email protected]>
Date:   Sat Sep 30 21:21:47 2023 +0400

    Update prover-market-page.mdx (#14854)

commit d9f4f43
Author: Marcus Wentz <[email protected]>
Date:   Fri Sep 29 21:40:08 2023 -0400

    docs(website): proposer + prover private key format and status log command to find errors (#14862)

    Co-authored-by: Roger <[email protected]>

commit f2ba221
Author: Razafindrabe Michaël Maminiaina <[email protected]>
Date:   Fri Sep 29 02:57:01 2023 +0300

    chore(docs): add RZF Node to prover list (#14847)

commit 7f4fe71
Author: Ravil Ayupov <[email protected]>
Date:   Thu Sep 28 08:14:01 2023 +0500

    docs(website): add prover endpoint to prover market page (#14841)

    Co-authored-by: dave | d1onys1us <[email protected]>

commit b3cd84a
Author: 0xCryptic <[email protected]>
Date:   Thu Sep 28 08:32:57 2023 +0530

    docs(website): Update enable-a-proposer.mdx with command to check Proposer logs (#14844)

commit db7c31d
Author: broccoin <[email protected]>
Date:   Thu Sep 28 00:26:00 2023 +0900

    fix(website): Correcting broken link (#14840)

commit 4cfcc42
Author: Kenk <[email protected]>
Date:   Wed Sep 27 09:35:38 2023 +0700

    feat(website): add izar and omnisea to ecosystem page (#14825)

    Co-authored-by: dave | d1onys1us <[email protected]>

commit f4b06db
Author: Kenk <[email protected]>
Date:   Tue Sep 26 21:00:19 2023 +0700

    feat(website): add gourds to ecosystem (#14818)

    Co-authored-by: dave | d1onys1us <[email protected]>

commit e704fc4
Author: moenorman <[email protected]>
Date:   Tue Sep 26 01:23:32 2023 -0400

    feat(website): add `ZK Nodes` to prover fee market (#14817)

commit 72e0431
Author: Marcus Wentz <[email protected]>
Date:   Mon Sep 25 22:38:12 2023 -0400

    docs(website): node troubleshooting info: `Skip an invlaid proposed transaction reason="nonce too low`  (#14814)

    Co-authored-by: Daniel Wang <[email protected]>
    Co-authored-by: Roger <[email protected]>

commit 73b2944
Author: dave | d1onys1us <[email protected]>
Date:   Mon Sep 25 22:14:51 2023 +0900

    docs(website): add windows guidance and remove sudo (#14806)

commit 1bc2ea7
Author: Marcus Wentz <[email protected]>
Date:   Mon Sep 25 09:14:39 2023 -0400

    feat(website): add Alphamint and Xverse to ecosystem (#14807)

commit bf92399
Author: Kenk <[email protected]>
Date:   Mon Oct 2 20:27:24 2023 +0700

    feat(website): add AIT Protocol to ecosystem (#14868)

commit a64ae4e
Author: dave <[email protected]>
Date:   Mon Oct 2 19:28:56 2023 +0900

    feat(website): add link to swap v3 (#14883)

commit 62719e7
Author: Kenk <[email protected]>
Date:   Mon Oct 2 15:09:45 2023 +0700

    feat(website): update TaikoL1 impl address (#14879)

commit d8f40c1
Author: xiaodino <[email protected]>
Date:   Sun Oct 1 21:56:48 2023 -0700

    chore(bridge-ui-v2): Update right top buttons to same UI styles (#14872)

commit 067c86a
Author: pessori55 <[email protected]>
Date:   Mon Oct 2 00:30:42 2023 +0900

    docs(website): add Sepolia faucet (#14865)

commit ff375f4
Author: Vitalik <[email protected]>
Date:   Sat Sep 30 21:21:47 2023 +0400

    Update prover-market-page.mdx (#14854)

commit 26adcd3
Author: Marcus Wentz <[email protected]>
Date:   Fri Sep 29 21:40:08 2023 -0400

    docs(website): proposer + prover private key format and status log command to find errors (#14862)

    Co-authored-by: Roger <[email protected]>

commit e900dd9
Author: Korbinian <[email protected]>
Date:   Fri Sep 29 23:14:11 2023 +0200

    chore(bridge-ui-v2): pre-select destination chain (#14850)

    Co-authored-by: David <[email protected]>

commit 745c196
Author: Kenk <[email protected]>
Date:   Fri Sep 29 22:30:17 2023 +0700

    feat(website): update Bitget on ecosystem page (#14855)

commit 9aec364
Author: Marcus Wentz <[email protected]>
Date:   Fri Sep 29 00:11:17 2023 -0400

    docs(website): prover proposer danger private key with nextra callout type error + generate isolated burner wallet (#14853)

commit eae3927
Author: Razafindrabe Michaël Maminiaina <[email protected]>
Date:   Fri Sep 29 02:57:01 2023 +0300

    chore(docs): add RZF Node to prover list (#14847)

commit 8dd8b61
Author: Korbinian <[email protected]>
Date:   Thu Sep 28 05:53:32 2023 +0200

    feat(bridge-ui-v2): NFT bridge stepper (#14811)

commit a61fd72
Author: Ravil Ayupov <[email protected]>
Date:   Thu Sep 28 08:14:01 2023 +0500

    docs(website): add prover endpoint to prover market page (#14841)

    Co-authored-by: dave | d1onys1us <[email protected]>

commit fac8f72
Author: 0xCryptic <[email protected]>
Date:   Thu Sep 28 08:32:57 2023 +0530

    docs(website): Update enable-a-proposer.mdx with command to check Proposer logs (#14844)

commit 7900d8e
Author: broccoin <[email protected]>
Date:   Thu Sep 28 00:26:00 2023 +0900

    fix(website): Correcting broken link (#14840)

commit 3bc0a45
Author: broccoin <[email protected]>
Date:   Wed Sep 27 21:36:55 2023 +0900

    fix(website): broken link (#14839)

    Co-authored-by: Daniel Wang <[email protected]>

commit cf4ebdd
Author: David Cardenas <[email protected]>
Date:   Wed Sep 27 05:35:01 2023 -0700

    docs(repo): Add YouTube to README (#14833)

    Co-authored-by: dave | d1onys1us <[email protected]>

commit c00a216
Author: dave | d1onys1us <[email protected]>
Date:   Wed Sep 27 21:33:26 2023 +0900

    docs(website): use remix by default (#14836)

commit 5942326
Author: dave | d1onys1us <[email protected]>
Date:   Wed Sep 27 15:12:20 2023 +0900

    docs(protocol): add taiko deployment docs and remove bll (#14834)

commit df2e8b9
Author: broccoin <[email protected]>
Date:   Wed Sep 27 11:39:28 2023 +0900

    docs(website): update node-runner-manual "how to compare .env and .env.sample" (#14831)

    Co-authored-by: dave | d1onys1us <[email protected]>

commit e7c66a3
Author: Kenk <[email protected]>
Date:   Wed Sep 27 09:36:05 2023 +0700

    feat(website): add proof market video (#14824)

    Co-authored-by: dave | d1onys1us <[email protected]>

commit be216d0
Author: Kenk <[email protected]>
Date:   Wed Sep 27 09:35:38 2023 +0700

    feat(website): add izar and omnisea to ecosystem page (#14825)

    Co-authored-by: dave | d1onys1us <[email protected]>

commit cb9c779
Author: Kenk <[email protected]>
Date:   Tue Sep 26 21:00:19 2023 +0700

    feat(website): add gourds to ecosystem (#14818)

    Co-authored-by: dave | d1onys1us <[email protected]>

commit 1405121
Author: moenorman <[email protected]>
Date:   Tue Sep 26 01:23:32 2023 -0400

    feat(website): add `ZK Nodes` to prover fee market (#14817)

commit 23f5bf6
Author: Marcus Wentz <[email protected]>
Date:   Mon Sep 25 22:38:12 2023 -0400

    docs(website): node troubleshooting info: `Skip an invlaid proposed transaction reason="nonce too low`  (#14814)

    Co-authored-by: Daniel Wang <[email protected]>
    Co-authored-by: Roger <[email protected]>

commit cde533c
Author: Marcus Wentz <[email protected]>
Date:   Mon Sep 25 22:13:06 2023 -0400

    docs(website): update FAQ `Does Taiko have a sequencer?` (#14812)

commit a67c495
Author: D <[email protected]>
Date:   Tue Sep 26 04:00:13 2023 +0200

    fix(protocol): Fix genesis tests (#14813)

commit 833b4f5
Author: dave | d1onys1us <[email protected]>
Date:   Mon Sep 25 22:15:22 2023 +0900

    chore(repo): update add_project issue template (#14805)

commit df7be21
Author: Kenk <[email protected]>
Date:   Mon Sep 25 20:15:12 2023 +0700

    feat(website): add uniswap v3 addresses (#14804)

    Co-authored-by: dave | d1onys1us <[email protected]>

commit 7367f01
Author: dave | d1onys1us <[email protected]>
Date:   Mon Sep 25 22:14:51 2023 +0900

    docs(website): add windows guidance and remove sudo (#14806)

commit e23c267
Author: Marcus Wentz <[email protected]>
Date:   Mon Sep 25 09:14:39 2023 -0400

    feat(website): add Alphamint and Xverse to ecosystem (#14807)

commit 23636c7
Author: Korbinian <[email protected]>
Date:   Mon Oct 2 16:09:49 2023 +0200

    bridgetabs sizes

commit 7fa506c
Author: Korbinian <[email protected]>
Date:   Wed Sep 27 17:01:06 2023 +0200

    import steps, validating ownership etc

commit 0bf4d78
Author: Korbinian <[email protected]>
Date:   Wed Sep 27 16:57:33 2023 +0200

    i18n

commit 02fac48
Author: Korbinian Kasberger <[email protected]>
Date:   Wed Sep 27 09:20:32 2023 +0200

    basic layout

commit e56a890
Author: Korbinian Kasberger <[email protected]>
Date:   Mon Sep 25 17:04:05 2023 +0200

    fix button size

commit 7616048
Author: Korbinian Kasberger <[email protected]>
Date:   Mon Sep 25 16:55:17 2023 +0200

     basic dynamic stepper

commit b9281d7
Author: Korbinian Kasberger <[email protected]>
Date:   Mon Sep 25 16:54:39 2023 +0200

    add bridgetypes, tabs to store
@KorbinianK
detectContract update and test
f984245
@KorbinianK
tokenInfo
d83bc54
@KorbinianK
check ownership
3f60b87
@KorbinianK
fix error for detectContractType
21c4664
@KorbinianK
test for getTokenInfo
ad26dbf
@KorbinianK
tokeninfo, form reset, processing fee and recipient
f129e4a
@KorbinianK
fixes
8047687
@KorbinianK
Merge branch 'main' into feat/bridge-ui-v2--nft-import-step
50e573d
@KorbinianK
fetching from eventIndexer
51ba505
@KorbinianK
todo texts
61e3c4d
@KorbinianK
basic layout setup and icons
a8426f9
@KorbinianK
safeReadContract util
fa7c78b
@KorbinianK
getTokenWithInfo util
6f09461
@KorbinianK
fetch from API, dummy image
ba27a4a
@KorbinianK
Merge remote-tracking branch 'origin/main' into feat/bridge-ui-v2--nf…
6b7610d 
…t-import-api
@KorbinianK
env fix
c92a0ae
@KorbinianK
Merge remote-tracking branch 'origin/feat/bridge-ui-v2--nft-import-ap…
3558dc5 
…i' into feat/bridge-ui-v2--review-step
@KorbinianK
group nfts by collection
12f1bde
@KorbinianK KorbinianK self-assigned this Oct 11, 2023
@vercel
Copy link

vercel bot commented Oct 11, 2023 •
edited
Loading

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

Name Status Preview Comments Updated (UTC)
bridge-ui-v2-a5 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 11, 2023 10:32pm
bridge-ui-v2-internal ✅ Ready (Inspect) Visit Preview Oct 11, 2023 10:32pm

@KorbinianK KorbinianK marked this pull request as ready for review October 11, 2023 22:31
@KorbinianK KorbinianK added this pull request to the merge queue Oct 12, 2023
Merged via the queue into main with commit c079223 Oct 12, 2023
7 of 8 checks passed
@KorbinianK KorbinianK deleted the feat/bridge-ui-v2--review-step branch October 12, 2023 09:11
@github-actions github-actions bot mentioned this pull request Oct 12, 2023
Merged
dionysuzx pushed a commit that referenced this pull request Oct 16, 2023
@KorbinianK @dionysuzx
feat(bridge-ui-v2): review step (#14940)
b894cd5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xiaodino xiaodino xiaodino approved these changes

@davidtaikocha davidtaikocha davidtaikocha approved these changes

@dantaik dantaik dantaik approved these changes

@dionysuzx dionysuzx Awaiting requested review from dionysuzx

@cyberhorsey cyberhorsey Awaiting requested review from cyberhorsey

Assignees

@KorbinianK KorbinianK

Labels
area.bridge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@KorbinianK @dantaik @davidtaikocha @xiaodino