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

Loans: specify & implement loan's modifications logic. #1195

Closed
lemunozm opened this issue Feb 7, 2023 · 2 comments
Closed

Loans: specify & implement loan's modifications logic. #1195

lemunozm opened this issue Feb 7, 2023 · 2 comments
Assignees
Labels
crcl-protocol Circle protocol related. I8-enhancement An additional feature. P5-soon Issue should be addressed soon. Q5-hard Can be done by an experienced coder with a good knowledge of the codebase.

Comments

@lemunozm
Copy link
Contributor

lemunozm commented Feb 7, 2023

Which part of the code is the issue addressing?

  • Feature

Description

The previous reprice method allows the modification of loan attributes instantly without any feedback to the investors. We want these changes to happen after certain time while the investors have feedback to decide if they want to keep their money there.

This forces to the loans pallet to act in the following way in order to modify certain critical loan attributes:

  • modify_intent: pallet-loans proposes to modify something, but the modification itself is locked
  • The pool unlocks that modification after some time.
  • modify: The pallet-loans is able to modify the loan's attributes

Research/based on

How will this affect the code base

  • Instead of reprice method. This modify allows for protecting investors first, giving clear feedback on what will be changed.

What are forseen obstacles or hurdles to overcome?

@lemunozm lemunozm added Q5-hard Can be done by an experienced coder with a good knowledge of the codebase. P7-asap Issue should be addressed in the next days. I8-enhancement An additional feature. crcl-protocol Circle protocol related. labels Feb 7, 2023
@lemunozm lemunozm self-assigned this Feb 14, 2023
@mustermeiszer
Copy link
Collaborator

Just for reference.

@lemunozm lemunozm added P5-soon Issue should be addressed soon. and removed P7-asap Issue should be addressed in the next days. labels Apr 11, 2023
@lemunozm
Copy link
Contributor Author

lemunozm commented Jun 9, 2023

Done in #1384

@lemunozm lemunozm closed this as completed Jun 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
crcl-protocol Circle protocol related. I8-enhancement An additional feature. P5-soon Issue should be addressed soon. Q5-hard Can be done by an experienced coder with a good knowledge of the codebase.
Projects
None yet
Development

No branches or pull requests

2 participants