Skip to content
This repository has been archived by the owner on Jun 24, 2022. It is now read-only.

Claim dynamic prices #2182

Merged
merged 11 commits into from Jan 18, 2022
Merged

Claim dynamic prices #2182

merged 11 commits into from Jan 18, 2022

Conversation

alfetopito
Copy link
Contributor

Summary

Loading prices dynamically from the contract

No functional changes, should behave the same
Prices should also be the same

Also added dependency on cow-token package and using ClaimType enum from there

@alfetopito alfetopito self-assigned this Jan 18, 2022
@alfetopito alfetopito requested a review from a team January 18, 2022 05:29
Copy link
Contributor

@anxolin anxolin left a comment

Choose a reason for hiding this comment

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

Great PR!

This makes it more flexible

@anxolin
Copy link
Contributor

anxolin commented Jan 18, 2022

I will merge, pls post-merge review pls

EDIT: I changed my mind, build error!

@github-actions
Copy link
Contributor

  • 🔭 GP Swap: Gnosis Protocol v2 Swap UI

@alfetopito alfetopito merged commit 1c15ca5 into claim Jan 18, 2022
@alfetopito alfetopito deleted the claim-dynamic-prices branch January 18, 2022 18:27
@anxolin anxolin mentioned this pull request Jan 19, 2022
maria-vslvn pushed a commit that referenced this pull request Jan 25, 2022
Co-authored-by: Ian Lapham <ianlapham@Ians-MacBook-Pro.local>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants