usecannon / cannon

Manage EVM protocol deployments. Inspired by Docker, Terraform, and npm.
https://usecannon.com
GNU General Public License v3.0
104 stars 21 forks source link

feat(website): CAN-479 - Refactor custom providers #1331

Closed nicosampler closed 2 months ago

nicosampler commented 2 months ago

This PR closes CAN-479. closes https://linear.app/usecannon/issue/CAN-473/ui-is-not-using-the-configured-providers-from-the-settings-page

I've converted the useProviders hook to a Provider which exposes useProviders.

Hopefully, this PR fixes the case of using custom providers not supported by viem.

changeset-bot[bot] commented 2 months ago

⚠️ No Changeset found

Latest commit: 180aeb4fa3ca090b3f5fd43ef98a6c8db277140b

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

nx-cloud[bot] commented 2 months ago

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 9da2d0b3d81682ebc1517572ff268a476e1a53a8. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 1 target - [`lerna run build --ignore cannon-website --ignore @usecannon/indexer --ignore repo --ignore @usecannon/api`](https://cloud.nx.app/runs/t6DvM9vDPw?utm_source=pull-request&utm_medium=comment)

Sent with 💌 from NxCloud.

dbeal-eth commented 2 months ago

@nicosampler please check if this PR replaces or should be in tandem or what with this PR https://github.com/usecannon/cannon/pull/1330

nicosampler commented 2 months ago

@nicosampler please check if this PR replaces or should be in tandem or what with this PR #1330

@dbeal-eth Ideally it should replace #1300, but I couldn't find the time yet if it works properly. Noah sent me a link to try to deploy to Snax.