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

Add cctp to explorer ui #1203

Merged
merged 13 commits into from
Aug 4, 2023
Merged

Add cctp to explorer ui #1203

merged 13 commits into from
Aug 4, 2023

Conversation

nautsimon
Copy link
Contributor

@nautsimon nautsimon commented Aug 1, 2023

Description
simple conditional text that appears on cctp bridge events + adding cctp token addresses, contract address, etc.

example:
Screen Shot 2023-08-01 at 10 02 32 AM

  • add cctp text
  • add token addresses
  • add contract addresses (make sure link works)
  • resolve sender nonsense

8a655d7935c89bff39d5522e14ae9e17a0cbf975: explorer-ui preview link
900a4908160de100631b86c3902deec477e78430: explorer-ui preview link
2726659e8f0f9420bac85e0d52d13dafeb7c944d: explorer-ui preview link
a9ee1690cf9f256bb923b836f9a30457b612459f: explorer-ui preview link
3e6182cebe7675a034c7e4167d2caa91b342403e: explorer-ui preview link
b51269ec6e46e6a4f181b15b8f574e2f30597535: explorer-ui preview link

@codecov
Copy link

codecov bot commented Aug 1, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (902c10b) 58.88689% compared to head (4a45c87) 58.88689%.
Report is 1 commits behind head on master.

Additional details and impacted files
@@              Coverage Diff              @@
##              master       #1203   +/-   ##
=============================================
  Coverage   58.88689%   58.88689%           
=============================================
  Files            533         533           
  Lines          32216       32216           
  Branches         276         276           
=============================================
  Hits           18971       18971           
  Misses         11545       11545           
  Partials        1700        1700           
Flag Coverage Δ
packages 82.77778% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

packages/explorer-ui/pages/tx/[kappa].tsx Outdated Show resolved Hide resolved
packages/explorer-ui/pages/tx/[kappa].tsx Outdated Show resolved Hide resolved
@github-actions github-actions bot added size/s and removed size/xs labels Aug 1, 2023
@nautsimon nautsimon requested a review from trajan0x August 1, 2023 16:11
@nautsimon nautsimon marked this pull request as ready for review August 1, 2023 16:11
Copy link
Contributor

@trajan0x trajan0x left a comment

Choose a reason for hiding this comment

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

LGTM

@trajan0x trajan0x mentioned this pull request Aug 4, 2023
@trajan0x
Copy link
Contributor

trajan0x commented Aug 4, 2023

Should be good to merge w/ #1234

@trajan0x trajan0x merged commit 9ab6502 into master Aug 4, 2023
38 checks passed
@trajan0x trajan0x deleted the feat/add-cctp-to-explorer-ui branch August 4, 2023 23:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants