-
Notifications
You must be signed in to change notification settings - Fork 5
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
feat: msp chargeable tick is current tick #234
base: main
Are you sure you want to change the base?
Conversation
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.
lgtm modulo @ffarall comment about the tests
I have now a failing test So when a dynamic stream payment is being charged the index price returned is the default value which is 0 and not the value from the ProviderLastChargeableInfo that has been inserted in the test. |
Last chargeable tick is set by default to be the current tick.
In case of dynamic payment we get the last chargeable info and get the last chargeable tick for it.