-
Notifications
You must be signed in to change notification settings - Fork 48
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
fix: add missing events to x/profiles #759
Merged
Merged
Changes from all commits
Commits
Show all changes
64 commits
Select commit
Hold shift + click to select a range
10efacc
added the Proto definitions of the subspaces types and messages
RiccardoM 32f7981
improved the Proto file definition
RiccardoM 9293ae2
improved the Proto file definition
RiccardoM ed8e474
started working on the implementation of the message server methods
RiccardoM b0f4f61
more work on the handling of messages
RiccardoM fca2aa9
added unit tests for subspaces and permissions
RiccardoM e08f3b3
completed the keeper package tests
RiccardoM 3fe18f4
completed tests for the types package
RiccardoM 5a3362f
Merge branch 'master' of github.com:desmos-labs/desmos into riccardo/…
RiccardoM 735a1fd
added the CLI tests
RiccardoM a4d22bd
added the integration tests
RiccardoM 78a5499
added changeset entry
RiccardoM 8233131
updated x/profiles to use the correct type for subspaces ids
RiccardoM 10e2a71
added initial subspaces id to the subspaces genesis
RiccardoM c7006d3
fixed simulation tests
RiccardoM 2b04da8
added changeset entry for x/profiles
RiccardoM d28fef6
fixed formatting
RiccardoM 5a557c3
added migration for both modules
RiccardoM 7d91aed
fixed lint
RiccardoM 4c62789
chore: updated Proto packages to match real versions
RiccardoM 1615ab3
chore: renamed subspace fields to subspace_id inside x/profiles Proto…
RiccardoM 5dd2122
small lint
RiccardoM e422c23
chore: replaced all expErr with shouldErr
RiccardoM 4fc9328
moved all the relationships-related things into the new x/relationshi…
RiccardoM ee244a9
Merge branch 'master' of github.com:desmos-labs/desmos into riccardo/…
RiccardoM d53accf
Updated Swagger definition
RiccardoM a85a164
fixed compilation errors
RiccardoM 93608a6
improving tests
RiccardoM b437666
Merge remote-tracking branch 'origin/riccardo/relationships-module' i…
RiccardoM 3f40b9e
fixed unit tests
RiccardoM c4e8f5e
Merge branch 'master' of github.com:desmos-labs/desmos into riccardo/…
RiccardoM 2521bf9
fixed simulation operations
RiccardoM df349dd
added the filtering for relationships during the migration
RiccardoM 3556916
updated to Cosmos v0.45.x and added a custom OrderMigrations
RiccardoM cc50c97
fixed tests
RiccardoM 86289a7
added missing profiles decoder tests
RiccardoM 8e6f763
added missing store tests
RiccardoM db121e9
added missing DONTCOVER directives
RiccardoM f4055b4
bring back build step for sims tests
RiccardoM 2110e42
increased liveliness test duration
RiccardoM bd102e3
use go 1.17 instead of 1.15
RiccardoM 4078c7b
small change to trigger tests
RiccardoM 9ef4cf6
added mint and authz to store keys
RiccardoM 4173742
improved the performance of user groups reading
RiccardoM 8dc035d
improved the subspace simulations operations
RiccardoM aa761b1
improved the performance to iterator-related methods inside the suspa…
RiccardoM b5d8e52
reduced liveliness test
RiccardoM 8e01fef
changed how Desmos is built for Linux by using a Docker container
RiccardoM d26d36c
fixed Docker
RiccardoM 654fe44
updated make-build
RiccardoM 9d2b3b9
trying to fix build-linux
RiccardoM 447a887
trying to fix build-linux
RiccardoM ffd76a3
trying to fix build-linux
RiccardoM 355b3a7
updated Proto field numberings
RiccardoM 7ee6fb7
fixed handler comment
RiccardoM cda19c2
added the subspace hook to delete relationships and blocks when a sub…
RiccardoM f91d6bc
fixed hooks tests
RiccardoM a46f4c3
Merge branch 'riccardo/fix-actions' of github.com:desmos-labs/desmos …
RiccardoM f4c1721
fix: added missing x/profiles events
RiccardoM c62adf3
fixed simulation tests
RiccardoM 82e9d2b
fixed simulation tests
RiccardoM eab8001
Merge branch 'riccardo/relationships-module' of github.com:desmos-lab…
RiccardoM f8558cc
added changeset entry
RiccardoM 41a193f
Merge branch 'master' of github.com:desmos-labs/desmos into riccardo/…
RiccardoM File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
6 changes: 6 additions & 0 deletions
6
.changeset/entries/0c6dff117a0e7eafc80aa5a3932b2f5456d2e7c4056492c123863d9ff82e07b3.yaml
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
type: fix | ||
module: x/profiles | ||
pull_request: 759 | ||
description: Added the emission of missing events | ||
backward_compatible: true | ||
date: 2022-02-28T13:18:32.965500736Z |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
This is to fix @bragaz comment on the relationships PR which I didn't address due to the automerge