Skip to content

refactor(subgraphs/exchange): fix build:fuse execution errors - #2

Open
diegofigs wants to merge 2 commits into
masterfrom
diegofigs/chore-update-exchange
Open

refactor(subgraphs/exchange): fix build:fuse execution errors#2
diegofigs wants to merge 2 commits into
masterfrom
diegofigs/chore-update-exchange

Conversation

@diegofigs

Copy link
Copy Markdown
Contributor

No description provided.

@diegofigs
diegofigs requested a review from mul53 April 11, 2022 12:39

@diegofigs diegofigs left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

self-review

"deploy:avax": "graph deploy --node https://api.thegraph.com/deploy/ --ipfs https://api.thegraph.com/ipfs/ traderjoe-xyz/exchange exchange.avax.yaml",
"deploy:rinkeby": "graph deploy --node https://api.thegraph.com/deploy/ --ipfs https://api.thegraph.com/ipfs/ traderjoe-xyz/exchange-rinkeby-ii exchange.rinkeby.yaml",
"deploy:fuse": "graph deploy --node https://api.thegraph.com/deploy/ --ipfs https://api.thegraph.com/ipfs/ voltfinance/voltage-exchange exchange.fuse.yaml",
"deploy:fuse": "graph deploy --access-token $GRAPH_TOKEN --node https://api.thegraph.com/deploy/ --ipfs https://api.thegraph.com/ipfs/ diegofigs/voltfinance-exchange exchange.fuse.yaml",

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

REMINDER: this should be changed back before merge

FACTORY_ADDRESS,
WHITELIST,
JOE_USDT_PAIR_ADDRESS,
VOLT_FUSD_PAIR_ADDRESS,

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

opted on using this pair since its the most liquid, regardless I think this reference is not used

* VOLT price is the weighted average of VOLT/WFUSE * WFUSE and VOLT/USDC.
*
*/
export function getWavgJoePrice(block: ethereum.Block = null): BigDecimal {

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Function was changed because build:fuse command failed reading a reference here; think this method is not called regardless.

@diegofigs diegofigs changed the title refactor(exchange): fix build:fuse execution errors refactor(subgraphs/exchange): fix build:fuse execution errors Apr 11, 2022
@mul53

mul53 commented Apr 21, 2022

Copy link
Copy Markdown
Contributor

@diegofigs i think we can also try to finish this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants