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

Add notifiers for price and liquidity changes in AMM #2458

Closed
katelynsills opened this issue Feb 17, 2021 · 2 comments
Closed

Add notifiers for price and liquidity changes in AMM #2458

katelynsills opened this issue Feb 17, 2021 · 2 comments
Assignees
@gibson042
Labels
AMM enhancement New feature or request good first issue Good for newcomers Inter-protocol Overarching Inter Protocol notifier Zoe Contract Contracts within Zoe
Milestone
Mainnet 1 RC0

Comments

@katelynsills
Copy link
Contributor

katelynsills commented Feb 17, 2021 •
edited by Chris-Hibbert
Loading

What is the Problem Being Solved?

multipool autoswap should use notifiers to publish changes so that other contracts can subscribe for updates.

Description of the Design

Security Considerations

Test Plan
@katelynsills katelynsills added the enhancement New feature or request label Feb 17, 2021
@Chris-Hibbert Chris-Hibbert changed the title We should add notifiers for price and liquidity changes. Add notifiers for price and liquidity changes. Feb 17, 2021
@Chris-Hibbert Chris-Hibbert mentioned this issue Feb 17, 2021
Closed
12 tasks
@Chris-Hibbert Chris-Hibbert changed the title Add notifiers for price and liquidity changes. Add notifiers for price and liquidity changes in multipoolAutoswap Feb 17, 2021
@Chris-Hibbert Chris-Hibbert added notifier Zoe Contract Contracts within Zoe labels Feb 17, 2021
@Chris-Hibbert Chris-Hibbert added this to the Beta Launch milestone Feb 17, 2021
@rowgraus rowgraus removed this from the Beta Initial Launch milestone Mar 29, 2021
@Chris-Hibbert Chris-Hibbert changed the title Add notifiers for price and liquidity changes in multipoolAutoswap Add notifiers for price and liquidity changes in AMM Jan 18, 2022
@Chris-Hibbert
Copy link
Contributor

Changed the title to reflect that this is still a valid request with respect to the newer AMM.

@Chris-Hibbert Chris-Hibbert added AMM good first issue Good for newcomers Inter-protocol Overarching Inter Protocol labels Jan 18, 2022
@Chris-Hibbert Chris-Hibbert self-assigned this Jan 18, 2022
@Tartuffo Tartuffo added MN-1 restival to be done before RUN Protocol Purple Team festival labels Jan 21, 2022
@turadg turadg assigned turadg and unassigned Chris-Hibbert Jan 24, 2022
@Tartuffo Tartuffo removed the MN-1 label Feb 7, 2022
@Tartuffo Tartuffo assigned gibson042 and unassigned turadg Feb 14, 2022
@Tartuffo Tartuffo removed the restival to be done before RUN Protocol Purple Team festival label Feb 14, 2022
@Tartuffo
Copy link
Contributor

Superseded by #4648

@Tartuffo Tartuffo added this to the Mainnet 1 milestone Mar 23, 2022
@Tartuffo Tartuffo modified the milestones: Mainnet 1, RUN Protocol RC0 Apr 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gibson042 gibson042

Labels
AMM enhancement New feature or request good first issue Good for newcomers Inter-protocol Overarching Inter Protocol notifier Zoe Contract Contracts within Zoe
Projects
None yet
Milestone
Mainnet 1 RC0
Development

No branches or pull requests
6 participants
@turadg @Tartuffo @gibson042 @katelynsills @Chris-Hibbert @rowgraus