Implement user blocking/unblocking flow #5
Conversation
There was a problem hiding this comment.
@Vladyslav-Kuksiuk please make sure to do the following things:
-
review this PR by yourself, and do that really-really thoroughly;
-
next time perform p.1 before submitting a PR.
| */ | ||
|
|
||
| /** | ||
| * Provides ChatSPN `User` commands and common commands interfaces. |
There was a problem hiding this comment.
We don't use back-ticks in Javadoc. We use {@code ...}.
And another thing is that I don't understand why you want to have a monospaced don't here.
Please review.
Same goes for the similar cases within this PR.
| @@ -45,3 +45,10 @@ message UserRegistered { | |||
| // A name of the registered user. | |||
| string name = 2 [(required) = true]; | |||
| } | |||
|
|
|||
| // A new user chats has been created. | |||
There was a problem hiding this comment.
This is hardly a piece of English.
Please do better.
|
|
||
| import "spine/core/user_id.proto"; | ||
|
|
||
| // A user profile view in the chatting system. |
There was a problem hiding this comment.
?
Please make sure to review your own work before submitting a PR.
Vladyslav-Kuksiuk
changed the title
UserChats creation flow
There was a problem hiding this comment.
@Vladyslav-Kuksiuk as discussed vocally, BlockUser does not to make much sense if sent to User aggregate, since it introduces some discrepancy (i.e. who blocks whom?). So please return to this PR later.
This PR adds an implementation of the user blocking and unblocking flows.
User blocking flow model:
User unblocking flow model:
