-
Notifications
You must be signed in to change notification settings - Fork 14
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
Index order events #145
Index order events #145
Conversation
needs integration tests + price |
CURRENT PROBLEM:
|
Here is the source code from the contracts which sets the |
we can still merge the PR as it is since it is working, without the more detailed things like the order schema or how we work with the db |
From what I've discussed with @alexcos20, the preferred schema was something that follows this model: {
id: txID,
type: 'startOrder' | 'reuseOrder'
timestamp: block.timestamp,
consumer,
payer,
startOrderId: (if type reuse)
} @alexcos20 can you confirm as well and review it? |
Fixes #125 .
Changes proposed in this PR: