-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add General Schema #6
Conversation
schemas/general/SCHEMA.md
Outdated
| signature | The signature of the airdrop. | string | | ||
| method_id | The method ID of the airdrop. | string | |
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.
what are these?
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.
Frankly, I am not totally sure. This showed up in the master schema, so I left it in there.
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.
this looks like the methodID of the first outgoing transactions. we can remove as well
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.
Okay, I removed method_id and signature
schemas/general/SCHEMA.md
Outdated
| pool_address | The smart contract address of the pool. | string | | ||
| pool_name | The name of the pool (ie, pool() in the smart contract, if it exists). | string | | ||
| total_value_locked_usd | The total value locked within this pool in USD. | number | | ||
| return_30d | The return of this pool over 30 days. | number | |
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.
checking with garrett for return_30d
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.
checked with garrett, we can remove return_30d
@melotik
tableName
s for the tables?