-
Notifications
You must be signed in to change notification settings - Fork 200
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
return promise from siweConfig.createMessage #324
Conversation
@anukul is attempting to deploy a commit to the Aave Team on Vercel. A member of the Team first needs to authorize it. |
cc @lochie |
@@ -21,7 +21,7 @@ export type SIWEConfig = { | |||
nonce: string; | |||
address: string; | |||
chainId: number; | |||
}) => string; | |||
}) => Promise<string>; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should allow for both string | Promise<string>
to avoid breaking changes
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
adding support for this in #387 |
* viem siwe * wagmi fix * cleanup * handle all chains * use parseSiweMessage * types * types * Update package.json * user defined chain * add public client * include type * use config * update packages * bump versions + fix types * update packages * better error information * fix merge * add async support ( #324 ) * versions * Update CHANGELOG.md * bump viem * bump * dep fix * fix testebench * deo
Related - #323