-
Notifications
You must be signed in to change notification settings - Fork 0
Description
What would you like us to improve?
Amine from Chainlink, has updated his PR to include live data fetching.
Please merge his PR in our PR and make sure things work as expected..
Things to note:
- Ensure that all pages show correct (no broken UI)
- Ensure that if a value is missing (that seems to be the case from his message) the UI is correct
- Fix possible merge conflict
- Review the UI and ensure all values show correctly
I’ve just updated this PR: smartcontractkit#3172 to use the recent changes from @Andre (now the docs vercel functions call AtlasDB to fetch some information in real time).
Can you please ask thisdot to use it then point me to their PR with my latest changes?From Chainlink Labs | Today at 7:23 PM[7:26 PM]Yeah, thanks - just please tell him to tag me when their rebase is finished, and I’ll review/test their PR (now that we are using real prod data, I don’t think we have v2 pools yet so that’s normal that we won’t be able to see every feature) , most importnat now is to make sure their code pulls data from where it’s supposed to pull data fromFrom Chainlink Labs | Today at 7:26 PM
How important it is?
important