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

Redemption interface update #39

Merged
merged 2 commits into from
Jan 25, 2024
Merged

Redemption interface update #39

merged 2 commits into from
Jan 25, 2024

Conversation

0xmichalis
Copy link
Member

@0xmichalis 0xmichalis commented Jan 25, 2024

Refactor redemption interface to support
multi-TCO2 inputs and restrict to a single
TCO2 at the same time. This should help
make the interface future-proof to support
multi-TCO2 redemptions in the future.

This is the first step towards addressing https://github.com/neutral-protocol/dynamic-fee-pools/issues/35.

Group the TCO2 and amount together since
they are logically connected.
Refactor redemption interface to support
multi-TCO2 inputs and restrict to a single
TCO2 at the same time. This should help
make the interface future-proof to support
multi-TCO2 redemptions in the future.
@0xmichalis 0xmichalis changed the title Interface update Redemption interface update Jan 25, 2024
Copy link
Contributor

@PawelTroka PawelTroka left a comment

Choose a reason for hiding this comment

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

looks good to me

@0xmichalis 0xmichalis merged commit f4abb88 into main Jan 25, 2024
1 check passed
@0xmichalis 0xmichalis deleted the interface-update branch January 25, 2024 11:28
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.

2 participants