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

Fixing a theme color and combining the hooks for tracking transfers #2909

Conversation

emreboga
Copy link
Collaborator

@emreboga emreboga commented Oct 31, 2024

There are two separate changes in this PR:

  • Combining the existing two hooks for tracking a transaction.
  • Fixing a theme color issue for circular progress in Review transaction and Redeem views. See the final looks for dark and light themes below.

Fixes #2850

screenshot_2024-10-30_at_9 09 29___pm_720

screenshot_2024-10-30_at_9 10 17___pm_720

Copy link

netlify bot commented Oct 31, 2024

👷 Deploy request for wormhole-connect pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 5b1a745

Copy link

netlify bot commented Oct 31, 2024

👷 Deploy request for wormhole-connect-mainnet pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 5b1a745

Copy link
Collaborator

@artursapek artursapek left a comment

Choose a reason for hiding this comment

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

thanks for going back and cleaning this up <3

@emreboga emreboga merged commit d220610 into wormhole-foundation:development Nov 1, 2024
8 checks passed
@emreboga emreboga deleted the emre/fix-theme-combine-hooks branch November 1, 2024 20:15
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.

Refactor back down to a single useTrackTransfer hook
2 participants