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

[Draft] Retire carbon on base #2370

Closed
wants to merge 34 commits into from
Closed

[Draft] Retire carbon on base #2370

wants to merge 34 commits into from

Conversation

Atmosfearful
Copy link
Collaborator

WIP

Copy link

vercel bot commented Jun 14, 2024

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

Name Status Preview Comments Updated (UTC)
base ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 11, 2024 9:21am
klimadao-app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 11, 2024 9:21am
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
carbon ⬜️ Ignored (Inspect) Visit Preview Jul 11, 2024 9:21am

Anchor,
BaseLogo,
LogoWithClaim,
Text,
Copy link
Collaborator

Choose a reason for hiding this comment

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

@Atmosfearful Hmm, oddly seeing the base build failing locally for me when importing components from @klimadao/lib. Seems to be throwing a type error in lib/utils/useProvider/index.tsx on the following - web3state.provider.provider.on("accountsChanged", handleAccountsChanged) - will investigate further.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I've added a quick hacky 'any' to the web3State types to get the base project building. Will continue investigating the above issue though. Weird it seems to be only triggering on the base branch though and none of the other projects in the monorepo.

@0xMakka
Copy link
Collaborator

0xMakka commented Jul 25, 2024

This branch is no longer relevant as the POC was merged in #2378

@0xMakka 0xMakka closed this Jul 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants