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

VAL-111 Defaults only apply FL to cover the loan principal #113

Merged
merged 1 commit into from
Dec 1, 2022
Merged

VAL-111 Defaults only apply FL to cover the loan principal #113

merged 1 commit into from
Dec 1, 2022

Conversation

ams9198
Copy link
Contributor

@ams9198 ams9198 commented Nov 30, 2022

No description provided.

uint256 amount,
uint256 outstandingLoss
);
event FirstLossApplied(address indexed loan, uint256 amount);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The last argument felt like more trouble than it's worth, especially given that these terms aren't so easily defined...

Copy link
Contributor

Choose a reason for hiding this comment

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

makes sense, could always query the loan for the outstanding debt anyway.

Copy link
Contributor

@bricestacey bricestacey left a comment

Choose a reason for hiding this comment

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

This is much easier to reason about.

uint256 amount,
uint256 outstandingLoss
);
event FirstLossApplied(address indexed loan, uint256 amount);
Copy link
Contributor

Choose a reason for hiding this comment

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

makes sense, could always query the loan for the outstanding debt anyway.

@ams9198 ams9198 merged commit 162155e into circlefin:master Dec 1, 2022
@ams9198 ams9198 deleted the VAL-111 branch December 1, 2022 14:20
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