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 community cdp repay debt proposal #1565

Merged
merged 7 commits into from
Apr 20, 2023

Conversation

pirtleshell
Copy link
Member

@pirtleshell pirtleshell commented Apr 18, 2023

Description

  • adds x/community proposal CommunityCDPRepayDebtProposal which repays debts owned by the community module with community module funds

Follow up PR (#1566) adds corresponding committee permission for submission via committee.

Note: Does not include custom CLI command.

Checklist

  • Changelog has been updated as necessary.

@pirtleshell pirtleshell changed the title Rp community cdp repay debt proposal feat: add community cdp repay debt proposal Apr 18, 2023
@pirtleshell pirtleshell force-pushed the rp-community-cdp-repay-debt-proposal branch from 30cd64a to afc38bb Compare April 19, 2023 00:25
app.distrKeeper,
hardKeeper,
&hardKeeper,
Copy link
Member Author

Choose a reason for hiding this comment

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

creating a public record of the fact that because this was originally passed as a copy, any positions it created never had claims instantiated (because the hooks do not get registered to the copy).
this may have consequences for the open lend position held by the community module.

Copy link
Member Author

Choose a reason for hiding this comment

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

confirmed with @rhuairahrighairidh any fixes needed for this would be in a follow up pr. commencing merge!

@pirtleshell pirtleshell merged commit c5f414e into master Apr 20, 2023
@pirtleshell pirtleshell deleted the rp-community-cdp-repay-debt-proposal branch April 20, 2023 21:13
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