-
Notifications
You must be signed in to change notification settings - Fork 2
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
Create group rebased #50
Conversation
Co-authored-by: Oskar Stark <[email protected]>
Phpdoc update (thanks to @OskarStark) better Example (thanks to @slunak)
@@ -30,6 +30,8 @@ class GroupsClient extends Client implements ClientInterface | |||
public const ACTION_ENABLE_USER = 'enable_user'; | |||
public const ACTION_RENAME_GROUP = 'rename'; | |||
public const ACTION_CREATE_GROUP = 'create'; | |||
public const ACTION_LIST_GROUPS = 'list'; |
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.
Private?
Sorry, I dont understand purpose of this PR... it is called Create Group Rebased, but includes functionality of previous PR with list groups functionality. |
I think we can close this PR and work in Just merge the master branch in your branch, @slunak can squash all the commits while merging |
No description provided.