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

feature/PP-204 #27

Merged
merged 5 commits into from
Aug 12, 2022
Merged

feature/PP-204 #27

merged 5 commits into from
Aug 12, 2022

Conversation

franciscotobar
Copy link
Contributor

What

  • Validation for smart wallet address to verify if it belongs to the EOA and its deployed

Why

  • To be able to import a smart wallet

Refs

@franciscotobar franciscotobar requested a review from a team August 1, 2022 18:16
Copy link
Collaborator

@antomor antomor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@franciscotobar Thank you for the changes. I've just left some minors.

src/Utils.ts Outdated Show resolved Hide resolved
test/ContractInteractor.test.ts Outdated Show resolved Hide resolved
test/ContractInteractor.test.ts Outdated Show resolved Hide resolved
Copy link
Collaborator

@antomor antomor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@antomor antomor merged commit a03ced8 into master Aug 12, 2022
@antomor antomor deleted the feature/PP-204 branch August 12, 2022 16:11
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.

3 participants