Skip to content
This repository has been archived by the owner on Sep 6, 2023. It is now read-only.

feat: add website url field for chains #463

Closed
wants to merge 3 commits into from
Closed

feat: add website url field for chains #463

wants to merge 3 commits into from

Conversation

leovct
Copy link

@leovct leovct commented Aug 4, 2023

Description

Add a new field called websiteUrl to the Chain type which references a website or a documentation related to the chain. For example, the Ethereum chain would have a reference to https://ethereum.org/.

Related to #462

Question: I tried to add a changeset by selecting @wagmi/chains and pressing enter but didn't suceed...

$ pnpm changeset
🦋  Which packages would you like to include? · No items were selected
🦋  error You must select at least one package to release
🦋  error (You most likely hit enter instead of space!)
🦋  Which packages would you like to include? … 
◯ changed packages
  ◯ @wagmi/chains
◯ unchanged packages
  ◯ @wagmi/connectors

Additional Information

Your ENS/address: leovct.eth

@changeset-bot
Copy link

changeset-bot bot commented Aug 4, 2023

⚠️ No Changeset found

Latest commit: ddac938

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

@leovct
Copy link
Author

leovct commented Aug 11, 2023

@jxom @tmm any thoughts?

@tmm
Copy link
Member

tmm commented Aug 11, 2023

Not sure this is something we want to add right now, but if we do in the future, will make sure you get credit for this contribution.

@tmm tmm closed this Aug 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants