Skip to content

added history provider, modal content visibility fix#40

Merged
poocart merged 6 commits intostagingfrom
feature/history-provider
Feb 15, 2024
Merged

added history provider, modal content visibility fix#40
poocart merged 6 commits intostagingfrom
feature/history-provider

Conversation

@poocart
Copy link
Contributor

@poocart poocart commented Feb 14, 2024

Description

  • Adds Account History provider and listener.
  • Updated Balances listener sleep time to make it based on real things (confirmed block time).
  • Updated bottom modals content visibility to remove sluggishness when animated.

How Has This Been Tested?

  • Unit tests and ran dapp locally.

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

@poocart poocart requested a review from IAmKio February 14, 2024 13:20
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Feb 14, 2024

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: d8bc470
Status: ✅  Deploy successful!
Preview URL: https://957c2cc3.x-e62.pages.dev
Branch Preview URL: https://feature-history-provider.x-e62.pages.dev

View logs

@github-actions github-actions bot temporarily deployed to Preview (feature/history-provider) February 14, 2024 13:25 Inactive
@github-actions github-actions bot temporarily deployed to Preview (feature/history-provider) February 15, 2024 10:54 Inactive
@github-actions github-actions bot temporarily deployed to Preview (feature/history-provider) February 15, 2024 11:20 Inactive
Copy link
Collaborator

@IAmKio IAmKio left a comment

Choose a reason for hiding this comment

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

LGTM! 🎉

@poocart poocart merged commit 076af57 into staging Feb 15, 2024
@poocart poocart deleted the feature/history-provider branch February 15, 2024 13:28
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.

2 participants