Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Pool statistics from the graph #288

Merged
merged 37 commits into from
Jan 21, 2021
Merged

Pool statistics from the graph #288

merged 37 commits into from
Jan 21, 2021

Conversation

mihaisc
Copy link
Contributor

@mihaisc mihaisc commented Dec 2, 2020 •
edited by kremalicious
Loading

Get pool statistics directly from the graph without connecting a wallet.

closes #284

@mihaisc
graphql
590f5db 
Signed-off-by: mihaisc <mihai.scarlat@smartcontrol.ro>
@mihaisc
ignore generated
20753e5 
Signed-off-by: mihaisc <mihai.scarlat@smartcontrol.ro>
@mihaisc
delete generated
5020c72 
Signed-off-by: mihaisc <mihai.scarlat@smartcontrol.ro>
@mihaisc
fix travis
e35c102 
Signed-off-by: mihaisc <mihai.scarlat@smartcontrol.ro>
@mihaisc
fix travis
053534a 
Signed-off-by: mihaisc <mihai.scarlat@smartcontrol.ro>
@mihaisc
fix fetch
6328363 
Signed-off-by: mihaisc <mihai.scarlat@smartcontrol.ro>
@mihaisc
fix travis
3b27b0c 
Signed-off-by: mihaisc <mihai.scarlat@smartcontrol.ro>
@mihaisc
fix fetch
3ee63c7 
Signed-off-by: mihaisc <mihai.scarlat@smartcontrol.ro>
@mihaisc
update readme
9e676ea 
Signed-off-by: mihaisc <mihai.scarlat@smartcontrol.ro>
@mihaisc mihaisc requested a review from kremalicious as a code owner December 2, 2020 13:08
@vercel
Copy link

vercel bot commented Dec 2, 2020 •
edited
Loading

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/oceanprotocol/market/orwd8q0yz
✅ Preview: https://market-git-feature-graphpoolstatistics.oceanprotocol.vercel.app

@mihaisc mihaisc marked this pull request as draft December 2, 2020 13:08
@mihaisc
pool creator liquidit& statistics
459787c 
Signed-off-by: mihaisc <mihai.scarlat@smartcontrol.ro>
@mihaisc
graph with the graph
c2e8336 
Signed-off-by: mihaisc <mihai.scarlat@smartcontrol.ro>
@mihaisc
cleanup
3cf71dc 
Signed-off-by: mihaisc <mihai.scarlat@smartcontrol.ro>
@kremalicious kremalicious mentioned this pull request Dec 9, 2020
Merged
@mihaisc
fix query
8693f3c 
Signed-off-by: mihaisc <mihai.scarlat@smartcontrol.ro>
@mihaisc
update poll interval
303d52d 
Signed-off-by: mihaisc <mihai.scarlat@smartcontrol.ro>
@mihaisc
update graph url
e9a3422 
Signed-off-by: mihaisc <mihai.scarlat@smartcontrol.ro>
@mihaisc
fix loading
23c7630 
Signed-off-by: mihaisc <mihai.scarlat@smartcontrol.ro>
@mihaisc
fix codegen camelcase
c256ef3 
Signed-off-by: mihaisc <mihai.scarlat@smartcontrol.ro>
@kremalicious kremalicious mentioned this pull request Jan 12, 2021
Closed
@kremalicious
Copy link
Contributor

all the graph (the visual one) rendering issues seem to be gone, at least in Chrome and Safari. Might be some chart.js update which fixed it

kremalicious
kremalicious approved these changes Jan 13, 2021
View reviewed changes
Copy link
Contributor

@kremalicious kremalicious left a comment

Choose a reason for hiding this comment

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

working great! Can be released once Rinkeby & Ropsten subgraphs are up again

@kremalicious kremalicious added the Status: Blocked Blocked by dependency, platform requirement, etc (add comment to detail the reason) label Jan 13, 2021
@kremalicious kremalicious mentioned this pull request Jan 13, 2021
Closed
@mihaisc mihaisc mentioned this pull request Jan 15, 2021
Merged
2 tasks
@kremalicious
Copy link
Contributor

all confirmed working in all supported networks, wohoo

@kremalicious kremalicious removed the Status: Blocked Blocked by dependency, platform requirement, etc (add comment to detail the reason) label Jan 21, 2021
@kremalicious kremalicious merged commit 2737693 into main Jan 21, 2021
@kremalicious kremalicious deleted the feature/graphPoolStatistics branch January 21, 2021 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kremalicious kremalicious kremalicious approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

query subgraph for all pool statistics (part of TheGraph work)
2 participants
@mihaisc @kremalicious