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

feat: add prices api #630

Merged
merged 13 commits into from
Jan 28, 2025
Merged

feat: add prices api #630

merged 13 commits into from
Jan 28, 2025

Conversation

0xAlunara
Copy link
Collaborator

@0xAlunara 0xAlunara commented Jan 24, 2025

Heavily refactored on the Curve Monitor repo to make it as suitable as possible for curve.fi repo.

Package isn't ready for public publishing to the npm registry yet, internal usage is more important in the short term.
It doesn't have to be perfect from the start, we can add and modify params and typing as we use it over time.

I have split the actual refactoring that makes use of this package up into multiple smaller PRs which makes testing and reviewing easier. In those same PRs I will also modify the package code such that suits our needs (think of stuff like function parameters and such).

As such, this PR only adds code without using it yet.

Copy link

vercel bot commented Jan 24, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
curve-dapp ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 27, 2025 5:02pm
curve-dapp-storybook ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 27, 2025 5:02pm

Copy link
Contributor

@DanielSchiavini DanielSchiavini left a comment

Choose a reason for hiding this comment

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

Sorry, I don't think we should add all this code to this repository.
If we want to use this code, it should either be a separate repository/npm package or we should only include the code that we use at that point

README.md Outdated Show resolved Hide resolved
packages/prices-api/package.json Show resolved Hide resolved
packages/prices-api/src/index.ts Outdated Show resolved Hide resolved
packages/prices-api/src/timestamp.ts Outdated Show resolved Hide resolved
packages/prices-api/src/solver/parsers.ts Outdated Show resolved Hide resolved
packages/prices-api/tsconfig.json Outdated Show resolved Hide resolved
Co-authored-by: Daniel Schiavini <daniel.schiavini@gmail.com>
@0xAlunara 0xAlunara merged commit 4341d30 into main Jan 28, 2025
5 checks passed
@0xAlunara 0xAlunara deleted the feat/prices-api branch January 28, 2025 12:55
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.

3 participants