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

CIP-???? | Collateral reward #217

Closed
wants to merge 1 commit into from
Closed

Conversation

SebastienGllmt
Copy link
Contributor

@SebastienGllmt SebastienGllmt commented Feb 10, 2022

This PR describes adding a scheme to transfer change in collateral to the reward account during phase-2 validation failure which removes the need for wallets to explicitly set collateral (which is bad UX) and improves safety of collateral at the same time.

This approach, however, has multiple problems (outlined in the CIP) and would probably be best combined with a more general discussion about reviving the idea of chimeric accounts (as it would make implementing this CIP a much more natural solution)

This PR is part of a series of different ways we can improve collateral handling in Cardano. You can find the three options below

Collateral Key
Collateral Output
Collateral Reward

@KtorZ
Copy link
Member

KtorZ commented Mar 1, 2022

@SebastienGllmt did you intend to deprecate / make obsolete that one?

@KtorZ KtorZ changed the title Collateral reward CIP-41? | Collateral reward Mar 17, 2022
@mangelsjover mangelsjover added the State: Likely Deprecated Close if confirmed deprecated (or long waiting). label Apr 12, 2022
@KtorZ KtorZ changed the title CIP-41? | Collateral reward CIP-0041? | Collateral reward May 11, 2022
@KtorZ KtorZ changed the title CIP-0041? | Collateral reward CIP-???? | Collateral reward Sep 27, 2022
@KtorZ KtorZ closed this Nov 30, 2022
@KtorZ
Copy link
Member

KtorZ commented Nov 30, 2022

Closing this now, doing some cleanup. It's been "likely deprecated" for a while, without any new activity. Feel free to re-open with new elements if needs be.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
State: Likely Deprecated Close if confirmed deprecated (or long waiting).
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants