Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Update the implementation to use providers for better code clarity.
Description
I was going through the docs to use it with ethers library... But, the implementation code was pretty unclear about whether to use
Provider
here or not. The earlier code, just used thecreateAppKit
function, and that's it, but it as I infered from other examples (wagmi, for instance), this value returned by the function needed to be stored asappkit
, in the context. In the next code block,AppKit
was clearly being imported the way a Provider is done, therefore it made sense to update the code to be more clear about using aContextProvider
here.I went on to create a provider, and tried using it on my end, and it worked.
Tests
I am a beginner, so I couldn't do the following tests as suggested by maintainers:
.mdx
files with changes.