-
Notifications
You must be signed in to change notification settings - Fork 41
feat: [WIP] 🍰 Implement JoinGroup
, GroupMember
, SwitchGroupMemberRole
Resolvers
#5194
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
Conversation
,
GroupMember` Resolvers
,
GroupMember` ResolversEnterGroup
, GroupMember
Resolvers
EnterGroup
, GroupMember
ResolversJoinGroup
, GroupMember
Resolvers
JoinGroup
, GroupMember
ResolversJoinGroup
, GroupMember
, SwitchGroupMemberRole
Resolvers
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.
Very good unit tests! Very detailed and good to read. I prefer to see the expected after the API call. I read the API call and the see the expected result. Now I see the expected result, then I see the API call and I have to scroll up again to see the expected.
We should discuss some details
🍰 Pullrequest
Implement
JoinGroup
,GroupMember
, andSwitchGroupMemberRole
resolvers.Issues
Todo
EnterGroup
toJoinGroup
SwitchGroupMemberRole
toChangeGroupMemberRole
ChangeGroupMemberRole
ChangeGroupMemberRole
ChangeGroupMemberRole
has "updatedAt" newer as "createdAt"
?public
andclosed
hidden
ChangeGroupMemberRole
by an owner (no admin)Wolle