Skip to content
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

PP-510: script for removing tokens #81

Merged
merged 3 commits into from
Oct 21, 2022

Conversation

ironFe93
Copy link
Contributor

What

script for removing tokens

Why

utility script

Refs

@ironFe93 ironFe93 requested a review from a team October 20, 2022 11:11
@ironFe93 ironFe93 self-assigned this Oct 20, 2022
@antomor antomor changed the title feat: script for removing tokens PP-510: script for removing tokens Oct 20, 2022
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.

@ironFe93 thank you for your changes, but I don't think that the script is going to work.
The getAcceptedTokens is a view function, so we should call it with call ( e.g.: const acceptedTokens = await smartWalletDeployVerifier.methods.getAcceptedTokens().call();), especially because if we use send we won't have the return values of the function called, but just the transaction receipt

await smartWalletDeployVerifier.methods.getAcceptedTokens().send({from: accounts[0]});

@antomor
Copy link
Collaborator

antomor commented Oct 20, 2022

This PR depends on #80

@ironFe93
Copy link
Contributor Author

ironFe93 commented Oct 20, 2022

@antomor it is done now

@ironFe93 ironFe93 requested a review from antomor October 20, 2022 17:06
@antomor
Copy link
Collaborator

antomor commented Oct 21, 2022

@antomor it is done now

@ironFe93 I found another small issue, and fixed it directly ;-)

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 63e3b61 into master Oct 21, 2022
@antomor antomor deleted the feature/PP-510/remove-tokens-script branch October 21, 2022 08:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants