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

unclear purpose of IPolicy.ClaimState.Declined #103

Open
rapidddenis opened this issue May 29, 2023 · 4 comments
Open

unclear purpose of IPolicy.ClaimState.Declined #103

rapidddenis opened this issue May 29, 2023 · 4 comments
Labels
wontfix This will not be worked on

Comments

@rapidddenis
Copy link

rapidddenis commented May 29, 2023

In https://github.com/etherisc/gif-contracts/blob/b58fd27a34452e062d10b175f3bdd7ee16b9d1a7/contracts/modules/PolicyController.sol

ClaimState.Declined have no usage outside of PolicyController where it influences nothing.

@matthiaszimmermann matthiaszimmermann added the wontfix This will not be worked on label May 31, 2023
@matthiaszimmermann
Copy link
Contributor

the semantics of declining a claim is a (non-parametric) product where the policy holder would be allowed to create a claim. in such a situation the insurer needs a way to flag invalid claims.

this is done via the ClaimState.Declined state and can be set via base contract Product._declineClaim in the gif-interface repository.

@rapidddenis
Copy link
Author

rapidddenis commented May 31, 2023

APPLIED-> CONFIRMED->CLOSED / APPLIED->DECLINED. No close is needed for declined claim? Two "end" states? If claim is closed after declining then anyone who is trying to see the whole picture will have to dig into the logs?

@matthiaszimmermann
Copy link
Contributor

yes there are two end states for claims. for applications there are even 3 end states.

to see the whole picture for a specific policy the InstanceSerive functions claims and getClaim may be used. corresponding functions are provided for payouts.

@rapidddenis
Copy link
Author

Current implementation requires declined claim to be closed, otherwise policy will not close. So declined is not the end state.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

2 participants