Skip to content

feat: minimum-commission-rate#138

Draft
Jonathansumner wants to merge 23 commits intominimum-commission-rate-featurefrom
feat/minimum-commission-rate
Draft

feat: minimum-commission-rate#138
Jonathansumner wants to merge 23 commits intominimum-commission-rate-featurefrom
feat/minimum-commission-rate

Conversation

@Jonathansumner
Copy link
Copy Markdown
Member

No description provided.

@Jonathansumner Jonathansumner self-assigned this Feb 3, 2023
@Jonathansumner Jonathansumner changed the title wip: initial push for pairing feat: initial push for pairing Feb 3, 2023
@Jonathansumner Jonathansumner changed the title feat: initial push for pairing feat: minimum-commission-rate Feb 3, 2023
@Jonathansumner Jonathansumner force-pushed the feat/minimum-commission-rate branch from 9372d62 to db28752 Compare May 26, 2023 16:06
@github-actions github-actions bot added C:x/params and removed C:CLI labels May 26, 2023
@github-actions github-actions bot added the C:CLI label Jun 1, 2023
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This needs a 2nd pair of eyes, I'd say

return
}

func (k Keeper) MinCommissionRate(ctx sdk.Context) (res *sdk.Dec) {
Copy link
Copy Markdown
Member Author

@Jonathansumner Jonathansumner Jun 8, 2023

Choose a reason for hiding this comment

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

min commission not present - test behaviour when not present, when params don't contain that value

Comment thread x/staking/types/errors.go
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Revert

@pbukva pbukva changed the base branch from main to minimum-commission-rate-feature June 8, 2023 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants