-
Notifications
You must be signed in to change notification settings - Fork 46
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
Profiles fixes #415
Profiles fixes #415
Conversation
fixed related tests updated docs
Codecov Report
@@ Coverage Diff @@
## master #415 +/- ##
=======================================
Coverage 72.44% 72.44%
=======================================
Files 66 66
Lines 3429 3429
=======================================
Hits 2484 2484
Misses 847 847
Partials 98 98
Continue to review full report at Codecov.
|
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.
Everything looks great. Could you please add a function that converts an old Profile
into a new one? We will need this because this changes will need to be applied into the currently running testnet, and we need such function to be run during the upgrade.
The best approach would be to follow Cosmos one as done here (just we don't need the queryServer
, we only need to perform the migration):
https://github.com/cosmos/cosmos-sdk/blob/master/x/auth/keeper/migrations.go
…0160 profile to the newer profile version
…eonardo/profiles-bugs-fixes � Conflicts: � CHANGELOG.md
@RiccardoM I've added a legacy test that should fit the purpose, it looks like you've said that Protobuf doesn't care about fields names. |
…legacy v0160 profile to the newer profile version" This reverts commit 55fdff0
Description
This PR fixes some bugs of the
x/profiles
module.Closes #413.
Closes #414.
Checklist
CHANGELOG.md
file.Files changed
in the Github PR explorer.