-
Notifications
You must be signed in to change notification settings - Fork 3
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
feat: RewardAllocation
accepts target denom
#376
Conversation
this enables the migration of legacy farming plans from x/farming.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work! LGTM. Can you please pull the latest commit from main
branch? I left a single comment that is for a reminder.
x/farm/client/cli/tx.go
Outdated
@@ -84,19 +85,25 @@ $ %s tx %s create-private-plan "New Farming Plan" 2022-01-01T00:00:00Z 2023-01-0 | |||
var rewardAllocs []types.RewardAllocation | |||
for _, arg := range args[3:] { | |||
// TODO: use strings.Cut with go 1.18 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As we just upgraded go version to 1.18, I think we can use strings.Cut
if you pull the latest main
branch. I am leaving this comment for a reminder. You can tackle this in a separate PR.
I am not too sure why, but I encounter an error when I try to start a local network using This is the error message that I encounter.
|
…arm-denom-reward-allocation
7063824
to
fe56f5d
Compare
This issue has been resolved after merging |
Description
RewardAllocation
in aPlan
can now specify targetDenom
.This change makes the migration of old farming plans from x/farming possible.
Tasks
References
Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.
docs/
) or specification (x/<module>/spec/
)godoc
comments.Files changed
in the Github PR explorerCodecov Report
in the comment section below once CI passes