-
-
Notifications
You must be signed in to change notification settings - Fork 204
Conversation
🦋 Changeset detectedLatest commit: 4389532 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
👍 Dependency issues cleared. Learn more about Socket for GitHub ↗︎ This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored. |
New dependencies detected. Learn more about Socket for GitHub ↗︎
|
Hey @goosewobbler, is anything pending or missing in this PR? Was wondering why it's not yet merged. |
@salil-naik Nothing missing as far as I can tell. I'm still awaiting feedback or merge from a maintainer. I don't actually work for Frame any more but happy to push this through as the likely alternative is that it will be abandoned. |
@goosewobbler If you would like to give me access to this fork, i would like to clean up the failing checks and the injection logic to get this PR to merge-able |
@dirtycajunrice added |
Appreciate the effort that went into this! We've been busy with other work so haven't had time to support new connectors. We are reworking how connectors are setup (directly in the wagmi repo instead of references, new API, etc.) so once that is done this can be created again. Stayed tuned to the wagmi discussion board for more info. |
Description
New connector for Frame - Uses injected provider if available, otherwise connects using
eth-provider
.Connector request: #265
Main repo PR: wevm/wagmi#2296
Additional Information