Skip to content

added balances listener, added temporary sepolia assets, other fixes&…#38

Merged
poocart merged 3 commits intostagingfrom
feature/balance-listener-provider
Feb 14, 2024
Merged

added balances listener, added temporary sepolia assets, other fixes&…#38
poocart merged 3 commits intostagingfrom
feature/balance-listener-provider

Conversation

@poocart
Copy link
Contributor

@poocart poocart commented Feb 7, 2024

Description

  • Added balance listeners.
  • Fixed Sepolia balances.

How Has This Been Tested?

  • Locally with tests and dapp.

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)

@cloudflare-workers-and-pages
Copy link

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

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: babdaf6
Status: ✅  Deploy successful!
Preview URL: https://92f5d5cc.x-e62.pages.dev
Branch Preview URL: https://feature-balance-listener-pro.x-e62.pages.dev

View logs

@github-actions github-actions bot temporarily deployed to Preview (feature/balance-listener-provider) February 7, 2024 22:15 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.

Looks good so far

"@emotion/react": "^11.11.3",
"@emotion/styled": "^11.11.0",
"@etherspot/transaction-kit": "^0.7.0",
"@etherspot/transaction-kit": "^0.7.2",
Copy link
Collaborator

Choose a reason for hiding this comment

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

@taylorferran just a note the version has changed

"@sentry/cli": "^2.21.5",
"@sentry/react": "^7.81.1",
"axios": "^1.6.5",
"axios": "^1.6.7",
Copy link
Collaborator

Choose a reason for hiding this comment

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

@taylorferran version change

"ethers": "^5.7.2",
"firebase": "^10.6.0",
"i18next": "^23.7.6",
"lodash": "^4.17.21",
Copy link
Collaborator

Choose a reason for hiding this comment

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

@taylorferran new package addition for developers


useEffect(() => {
let expired = false;
let timeout: NodeJS.Timeout;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Had to look twice at this one 😂

@poocart poocart merged commit d3b0dd8 into staging Feb 14, 2024
@poocart poocart deleted the feature/balance-listener-provider branch February 14, 2024 12:36
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