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.

Sign up for GitHub

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

Jump to bottom

cli: replace program.rpc with program.methods in mocha test template #1531

Closed
wants to merge 1 commit into from
Closed

cli: replace program.rpc with program.methods in mocha test template #1531

wants to merge 1 commit into from

Conversation

NBNARADHYA
Copy link
Contributor

@NBNARADHYA NBNARADHYA commented Feb 28, 2022 •
edited
Loading

Signed-off-by: Nikhil B N nikhilbn365@gmail.com

Fixes #1463

program.rpc will be deprecated eventually

armaniferrante
armaniferrante reviewed Feb 28, 2022
View reviewed changes
cli/src/template.rs Outdated Show resolved Hide resolved
@NBNARADHYA
replace program.rpc with program.methods
0c02d7e 
Signed-off-by: Nikhil B N <nikhilbn365@gmail.com>
@paul-schaaf
Copy link
Contributor

hey @NBNARADHYA do you have time to finish this or would you like me to take over?

@paul-schaaf paul-schaaf mentioned this pull request Apr 4, 2022
Merged
@paul-schaaf
Copy link
Contributor

closed by #1732

@paul-schaaf paul-schaaf closed this Apr 4, 2022
@NBNARADHYA
Copy link
Contributor Author

hey @NBNARADHYA do you have time to finish this or would you like me to take over?

@paul-schaaf I apologise for not replying --> Was having exams.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@armaniferrante armaniferrante armaniferrante left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

cli: mocha test template should use program.methods not program.rpc
3 participants
@NBNARADHYA @paul-schaaf @armaniferrante