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.

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 DapiDataRegistry and use it to fetch active dAPIs #44

Merged
merged 6 commits into from
Oct 29, 2023
Merged

Conversation

Siegrift
Copy link
Collaborator

@Siegrift Siegrift commented Oct 26, 2023

Closes #35

Rationale

Because the contract is still in a PR, I've added a branch dependency and setup hardhat to build Typechain types and factories. These are then used in the actual implementation.

You need to run pnpm i && pnpm run build to install missing dependencies and build the Typechain artifacts.

@Siegrift Siegrift self-assigned this Oct 26, 2023
.eslintcache

# Hardhat files
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

These were generated by hardhat --init. I am not sure if all of them are needed, but it's better to keep them all.

README.md Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
src/config/schema.ts Outdated Show resolved Hide resolved
@bbenligiray
Copy link
Member

FYI we're still messing with the interface

Copy link
Collaborator

@aquarat aquarat left a comment

Choose a reason for hiding this comment

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

I can't fault it, this looks very cool and I learnt a few things like satisfies :D

🚀

README.md Outdated Show resolved Hide resolved
@Siegrift
Copy link
Collaborator Author

FYI we're still messing with the interface

I know. The solution is temporary but the current interface is better than the statically mocked data we have in the past. I don't see issues with updating the logic when the contract changes.

@Siegrift Siegrift force-pushed the create-data-feed-update-loop branch 3 times, most recently from dec8cc1 to dd44ad4 Compare October 27, 2023 13:20
Base automatically changed from create-data-feed-update-loop to main October 27, 2023 13:30
@Siegrift Siegrift merged commit beb66c8 into main Oct 29, 2023
4 checks passed
@Siegrift Siegrift deleted the add-contract branch October 29, 2023 08:56
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.

Fetch active dAPIs from DapiDataRegistry contract
3 participants