-
Notifications
You must be signed in to change notification settings - Fork 4
Adding incentive allocation to schema. #63
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
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -21,7 +21,7 @@ | |||||
| "transaction_hash", | ||||||
| "log_index" | ||||||
| ], | ||||||
| "description": "Transactional data on user level incentives claimed data.", | ||||||
| "description": "Pool level allocations for an incentive program period.", | ||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This table looks like transactional data to me, not pool level
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Just flipflopped the descriptions
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. this is pool level, this is the schema for our allocations, not really related to DOP, just wanted to make sure this is maintained somewhere |
||||||
| "properties": { | ||||||
| "timestamp": { | ||||||
| "description": "The timestamp of the claim.", | ||||||
|
|
@@ -65,6 +65,69 @@ | |||||
| } | ||||||
| } | ||||||
| }, | ||||||
| { | ||||||
| "label": "Pool Level Incentive Allocations", | ||||||
| "tableName": "pool_level_incentive_allocations", | ||||||
| "aggregation": "general_pools", | ||||||
| "uniqueKey": [ | ||||||
| "pool_address" | ||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We don't really use this right now, but to be proper the unique key should be pool_address and a timestamp field. You could add |
||||||
| ], | ||||||
| "description": "Transactional data on user level incentives claimed data.", | ||||||
| "properties": { | ||||||
| "reward_token": { | ||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
| "description": "The token address used to provide reward incentives.", | ||||||
| "type": "string" | ||||||
| }, | ||||||
| "token_symbol": { | ||||||
| "description": "The token symbol used to provide reward incentives.", | ||||||
| "type": "string" | ||||||
| }, | ||||||
| "amount": { | ||||||
| "description": "The amount of incentives provided over the period.", | ||||||
| "type": "double" | ||||||
| }, | ||||||
| "start_timestamp": { | ||||||
| "description": "The starting timestamp of the incentive period.", | ||||||
| "type": "timestamp" | ||||||
| }, | ||||||
| "end_timestamp": { | ||||||
| "description": "The ending timestamp of the incentive period.", | ||||||
| "type": "timestamp" | ||||||
| }, | ||||||
| "protocol_category": { | ||||||
| "description": "The category of the protocol (DEX, Lending, etc...).", | ||||||
| "type": "string" | ||||||
| }, | ||||||
| "protocol_name": { | ||||||
| "description": "The name of the protocol receiving incentives.", | ||||||
| "type": "string" | ||||||
| }, | ||||||
| "pool_address": { | ||||||
| "description": "The address of the pool receiving incentives.", | ||||||
| "type": "string" | ||||||
| }, | ||||||
| "reward_hash": { | ||||||
| "description": "The transaction hash of the rewards (if applicable).", | ||||||
| "type": "string" | ||||||
| }, | ||||||
| "log_index": { | ||||||
| "description": "The log index for the event in the rewards transaction (if applicable).", | ||||||
| "type": "int" | ||||||
| }, | ||||||
| "submission_timestamp": { | ||||||
| "description": "The timestamp of reward submissions.", | ||||||
| "type": "timestamp" | ||||||
| }, | ||||||
| "chain_id": { | ||||||
| "description": "The standard chain id.", | ||||||
| "type": "int" | ||||||
| }, | ||||||
| "program_name": { | ||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Usually this is something we add in after the fact, in the DOP pipeline
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. this is for non DOP incentives, this is for our allocations at the pool level, so this will come out of a DBT model (or have to be added manually, I am mainly keeping it here for our records) |
||||||
| "description": "The name of the program providing incentives.", | ||||||
| "type": "string" | ||||||
| } | ||||||
| } | ||||||
| }, | ||||||
| { | ||||||
| "label": "Airdrop", | ||||||
| "tableName": "general_airdrop", | ||||||
|
|
@@ -335,4 +398,4 @@ | |||||
| } | ||||||
| } | ||||||
| ] | ||||||
| } | ||||||
| } | ||||||
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.
Need to make this change in the schema.json, then autogen the md file
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.
Oh I see, this was just from autogenerating, ignore my comment then