-
Notifications
You must be signed in to change notification settings - Fork 102
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
feat: add sanity check on provider connect for clearer error message #181
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
d00e59a
to
f993fb7
Compare
bc2e02b
to
f2717f8
Compare
f993fb7
to
ecf61aa
Compare
8ff1972
to
1b90588
Compare
ecf61aa
to
5c36999
Compare
moldy530
reviewed
Oct 30, 2023
avasisht23
reviewed
Oct 30, 2023
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.
few comments/suggestions
68628f2
to
a05006f
Compare
c8f5c1d
to
9194213
Compare
This was referenced Oct 31, 2023
…or entry point address iff optional entry point param is given on init
9194213
to
bb048fd
Compare
avasisht23
approved these changes
Oct 31, 2023
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 looks good. few nits, then 🚢
Co-authored-by: Ajay Vasisht <43521356+avasisht23@users.noreply.github.com>
Co-authored-by: Ajay Vasisht <43521356+avasisht23@users.noreply.github.com>
Co-authored-by: Ajay Vasisht <43521356+avasisht23@users.noreply.github.com>
Co-authored-by: Ajay Vasisht <43521356+avasisht23@users.noreply.github.com>
Co-authored-by: Ajay Vasisht <43521356+avasisht23@users.noreply.github.com>
avasisht23
added a commit
that referenced
this pull request
Nov 3, 2023
…181) Co-authored-by: Ajay Vasisht <43521356+avasisht23@users.noreply.github.com>
avasisht23
added a commit
that referenced
this pull request
Nov 3, 2023
…181) Co-authored-by: Ajay Vasisht <43521356+avasisht23@users.noreply.github.com>
avasisht23
added a commit
that referenced
this pull request
Nov 3, 2023
…181) Co-authored-by: Ajay Vasisht <43521356+avasisht23@users.noreply.github.com>
Merged
moldy530
pushed a commit
that referenced
this pull request
Nov 10, 2023
…181) Co-authored-by: Ajay Vasisht <43521356+avasisht23@users.noreply.github.com>
moldy530
pushed a commit
that referenced
this pull request
Nov 10, 2023
…181) Co-authored-by: Ajay Vasisht <43521356+avasisht23@users.noreply.github.com>
avasisht23
added a commit
that referenced
this pull request
Nov 14, 2023
…181) Co-authored-by: Ajay Vasisht <43521356+avasisht23@users.noreply.github.com>
avasisht23
added a commit
that referenced
this pull request
Nov 14, 2023
…181) Co-authored-by: Ajay Vasisht <43521356+avasisht23@users.noreply.github.com>
avasisht23
added a commit
that referenced
this pull request
Nov 14, 2023
…181) Co-authored-by: Ajay Vasisht <43521356+avasisht23@users.noreply.github.com>
avasisht23
added a commit
that referenced
this pull request
Nov 14, 2023
…181) Co-authored-by: Ajay Vasisht <43521356+avasisht23@users.noreply.github.com>
denniswon
added a commit
that referenced
this pull request
Nov 16, 2023
…181) Co-authored-by: Ajay Vasisht <43521356+avasisht23@users.noreply.github.com>
denniswon
added a commit
that referenced
this pull request
Nov 16, 2023
…181) Co-authored-by: Ajay Vasisht <43521356+avasisht23@users.noreply.github.com>
moldy530
pushed a commit
that referenced
this pull request
Nov 16, 2023
…181) Co-authored-by: Ajay Vasisht <43521356+avasisht23@users.noreply.github.com>
moldy530
pushed a commit
that referenced
this pull request
Nov 16, 2023
…181) Co-authored-by: Ajay Vasisht <43521356+avasisht23@users.noreply.github.com>
avasisht23
added a commit
that referenced
this pull request
Dec 3, 2023
…181) Co-authored-by: Ajay Vasisht <43521356+avasisht23@users.noreply.github.com>
avasisht23
added a commit
that referenced
this pull request
Dec 3, 2023
…181) Co-authored-by: Ajay Vasisht <43521356+avasisht23@users.noreply.github.com>
avasisht23
added a commit
that referenced
this pull request
Dec 3, 2023
…181) Co-authored-by: Ajay Vasisht <43521356+avasisht23@users.noreply.github.com>
avasisht23
added a commit
that referenced
this pull request
Dec 3, 2023
…181) Co-authored-by: Ajay Vasisht <43521356+avasisht23@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
app.asana.com/0/1205598840815267/1205773893380310/f
Per the task linked above, added a sanity check early on on the current
connect()
so that users can get clear error message about the issue.Follow up PR will made to make entry point contract address param as optional for SmartContractAccount
Also, updated the doc for the missing/added
Other
methods for SCA class.PR-Codex overview
Detailed summary:
This PR introduces the
getEntryPointAddress
method to theISmartContractAccount
interface and theBaseSmartContractAccount
class. It also adds implementations and documentation for the method in various files. Additionally, it updates theSmartAccountProvider
class to use the connected account's entry point address or the default entry point address for the chain.