-
Notifications
You must be signed in to change notification settings - Fork 3
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
Initial draft for alpha release #1
Conversation
Use config object instead of env Improvements to Account
src/payment_channel_management_strategies/PaymentChannelManagementStrategy.js
Outdated
Show resolved
Hide resolved
… offset and call allowance
@@ -0,0 +1,61 @@ | |||
import { find } from 'lodash'; | |||
|
|||
export default class DefaultPaymentChannelManagementStrategy { |
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.
I would also add interface to implement custom strategy - not sure if interfaces are common in JS, thogh :-)
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.
Yeah, this would be communicated with api docs which would be made available soon.
6e01386
to
bf675e7
Compare
Listing things that need to be improved or addressed
I have added a simple example file which tries to invoke example calculator service. code.Sample usage has been added to snet-code-examples which can be easily tried out locally.
Need help with identifying the correctness of all the operations and corner cases if I have missed any. I feel we can do better with the design and happy to hear suggestions if any.