-
Notifications
You must be signed in to change notification settings - Fork 15
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
Groundhog Proposal #6
Conversation
* @param bytes _subscriptionHash is the identifier of the customer's subscription with its relevant details. | ||
* @return ( | ||
address destination, | ||
address recipient, |
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.
should we add token_address
and token_decimals
as return items to this array?
|
||
/** @dev returns the value of the subscription | ||
* @param bytes _subscriptionHash is the identifier of the customer's subscription with its relevant details. | ||
* @return value is the value that has been subscribed for the requested subscription |
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.
in what denomination?
```SOLIDITY | ||
|
||
/** @dev handles the incoming data to define subscription rules | ||
* @param address _destination |
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 think that _data
, and _meta
needs more specification to say what it is. what should be stored here?
for expires
and _interval
, are these unix timestamps and ints that represent seconds, respectively?
just read this end to end. looking good. my high level thoughts:
Low level thoughts?
|
No description provided.