Open insumity opened 1 week ago
x/ccv/provider/keeper/msg_server.go (1)
Pattern `**/*.go`: Review the Golang code for conformity with the Uber Golang style guide, highlighting any deviations.
.changelog/unreleased/api-breaking/provider/1998-change-ux-in-key-assignment.md (1)
`1-3`: **Changelog entry is clear and concise.** The changelog entry effectively summarizes the changes made in PR #1998, specifically the change in UX for key assignment..changelog/unreleased/state-breaking/provider/1998-change-ux-in-key-assignment.md (1)
`1-3`: **Changelog entry is clear and concise.** The changelog entry effectively summarizes the state-breaking changes made in PR #1998, specifically the change in UX for key assignment.x/ccv/provider/keeper/msg_server.go (1)
`57-64`: **Update in logging and event attributes is appropriate.** The change from logging the `consumerTMPublicKey` to `msg.ConsumerKey` aligns with the PR's objective to enhance clarity in log entries. The event attributes have also been updated accordingly.
Description
Modifies the user experience (UX) of key assignment by reverting #1732. Note that this is not a full revert of #1732 because #1732 added tests that made sense to keep here as well.
Before, when a validator tried to re-use a previously assigned consumer key that they used, we would return success and the assignment would be a no-op. With this PR, we return an error instead. The reason for this change was that we had a validator re-assigning an old consumer key they used but because the assignment was a no-op, the old consumer key was not set. As a result, the validator was confused on why the assignment was successful but the consumer key on the consumer chain remained unchanged.
Author Checklist
All items are required. Please add a note to the item if the item is not applicable and please add links to any relevant follow up issues.
I have...
!
to the type prefix if the change is state-machine breakingCHANGELOG.md
Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add your handle next to the items reviewed if you only reviewed selected items.
I have...
!
the type prefix if the change is state-machine breakingSummary by CodeRabbit
Bug Fixes
Tests
Documentation