ockam icon indicating copy to clipboard operation
ockam copied to clipboard

refactor(rust): make it easier to write commands' api req/res handlers

Open adrianbenavides opened this issue 2 years ago • 0 comments

Add an abstraction layer on top of connect_to to make it easier to write commands' api req/res handlers. It removes a lot of boilerplate around calling the background node and especially, around handling errors. This can prove especially helpful for new contributors.

adrianbenavides avatar Aug 08 '22 11:08 adrianbenavides

@adrianbenavides I really like where you're going with this PR 👍

mrinalwadhwa avatar Aug 14 '22 17:08 mrinalwadhwa

+1 Looks good to me! I just left some minor suggestions.

Thanks for all the comments! I've fixed all you pointed out in my last commit.

adrianbenavides avatar Aug 16 '22 11:08 adrianbenavides

@Mergifyio refresh

mrinalwadhwa avatar Aug 16 '22 22:08 mrinalwadhwa

refresh

✅ Pull request refreshed

mergify[bot] avatar Aug 16 '22 22:08 mergify[bot]