Skip to content
This repository has been archived by the owner on Dec 17, 2024. It is now read-only.

draft-delegator-incentive-contract #90

Merged
merged 16 commits into from
Nov 21, 2022
Merged

Conversation

kaisbaccour
Copy link
Contributor

No description provided.

Copy link
Collaborator

@webmaster128 webmaster128 left a comment

Choose a reason for hiding this comment

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

Great stuff! Let's get this done first

contracts/nois-delegator/src/contract.rs Outdated Show resolved Hide resolved
contracts/nois-delegator/src/contract.rs Outdated Show resolved Hide resolved
contracts/nois-delegator/src/contract.rs Outdated Show resolved Hide resolved
contracts/nois-delegator/src/contract.rs Outdated Show resolved Hide resolved
contracts/nois-oracle/src/state.rs Outdated Show resolved Hide resolved
contracts/nois-oracle/src/msg.rs Outdated Show resolved Hide resolved
contracts/nois-oracle/src/contract.rs Show resolved Hide resolved
.amount;
let bot_desired_incentive = incentive_amount(&config);
attributes.push(Attribute::new(
"bot_incentive",
bot_desired_incentive.to_string(),
));
if contract_balance > bot_desired_incentive.amount {
if delegator_contract_balance > bot_desired_incentive.amount {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Instead of doing this check, we could make a message that does not fail when the delegator does not have enough balance.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not fully sure how to do or if I understand correctly. if you could commit the change would be nice

contracts/nois-delegator/src/msg.rs Show resolved Hide resolved
contracts/nois-delegator/src/contract.rs Outdated Show resolved Hide resolved
@webmaster128
Copy link
Collaborator

Great work!

@webmaster128 webmaster128 merged commit d94fa9a into main Nov 21, 2022
@webmaster128 webmaster128 deleted the delegator-incentive-contract branch November 21, 2022 22:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants