-
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
Reaction registration implementation #120
Reaction registration implementation #120
Conversation
…dency between reactions and post modules
…reactions-registration-limiting-impl � Conflicts: � x/posts/alias.go � x/posts/internal/types/genesis.go
…ted new msgRegisterReaction
…ests, reaction test
added integration test + test helpers updated CHANGELOG.md TODO: - update migrate - update docs - implement unicode checks
- updated docs TODO: - update migrate - update PostReaction keeper's method
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.
LGTM overall. There are only some small refactoring errors here and there, and a couple of tests I would like to see being added to make sure that emojis are properly supported
@@ -508,3 +522,54 @@ func TestDesmosCLIPostsReactions(t *testing.T) { | |||
|
|||
f.Cleanup() | |||
} | |||
|
|||
func TestDesmosCLIRegisterReaction(t *testing.T) { |
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.
Can we have a test using an emoji as value such as 🎉 ?
…limiting-impl' into leonardo/reactions-registration-limiting-impl # Conflicts: # docs/types/reaction.md
…reactions-registration-limiting-impl � Conflicts: � CHANGELOG.md � cli_test/cli_posts_test.go
Co-Authored-By: Bot from GolangCI <42910462+golangcibot@users.noreply.github.com>
…limiting-impl' into leonardo/reactions-registration-limiting-impl
Codecov Report
@@ Coverage Diff @@
## master #120 +/- ##
==========================================
+ Coverage 82.56% 83.67% +1.11%
==========================================
Files 34 39 +5
Lines 1537 1654 +117
==========================================
+ Hits 1269 1384 +115
- Misses 238 239 +1
- Partials 30 31 +1 |
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.
LGTM
Description
This PR added the possibility to register a reaction to use it later when reacting to a post.
Closes #94
Checklist
CHANGELOG.md
file.Files changed
in the Github PR explorer.