Skip to content
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 new group with the Example and Test #27

Merged
merged 15 commits into from
Aug 8, 2024

Conversation

Vitexus
Copy link
Contributor

@Vitexus Vitexus commented Aug 3, 2024

No description provided.

@slunak
Copy link
Owner

slunak commented Aug 3, 2024

Thanks @Vitexus , I will review later today or tomorrow.

@slunak slunak self-requested a review August 3, 2024 18:54
@slunak slunak linked an issue Aug 3, 2024 that may be closed by this pull request
@Vitexus
Copy link
Contributor Author

Vitexus commented Aug 3, 2024

Thanks @Vitexus , I will review later today or tomorrow.

Слава Україні!

src/Client/Response/CreateGroupResponse.php Outdated Show resolved Hide resolved
src/Client/Response/CreateGroupResponse.php Outdated Show resolved Hide resolved
src/Api/Groups/Group.php Outdated Show resolved Hide resolved
src/Api/Groups/Group.php Outdated Show resolved Hide resolved
tests/Api/Groups/GroupTest.php Outdated Show resolved Hide resolved
tests/Api/Groups/GroupTest.php Outdated Show resolved Hide resolved
Co-authored-by: Oskar Stark <oskarstark@googlemail.com>
@slunak
Copy link
Owner

slunak commented Aug 3, 2024

@OskarStark I will do php-cs-fixer src after this PR.

Vitexus and others added 7 commits August 3, 2024 21:49
Co-authored-by: Oskar Stark <oskarstark@googlemail.com>
Co-authored-by: Oskar Stark <oskarstark@googlemail.com>
Co-authored-by: Oskar Stark <oskarstark@googlemail.com>
Co-authored-by: Oskar Stark <oskarstark@googlemail.com>
Co-authored-by: Oskar Stark <oskarstark@googlemail.com>
The Group Name now contains date & time to avoid conflict with already existing group.
@slunak slunak requested a review from OskarStark August 4, 2024 08:14
Co-authored-by: Oskar Stark <oskarstark@googlemail.com>
@slunak
Copy link
Owner

slunak commented Aug 7, 2024

@OskarStark are you happy with your changes requested?

src/Api/Groups/Group.php Outdated Show resolved Hide resolved
src/Api/Groups/Group.php Outdated Show resolved Hide resolved
Vitexus and others added 2 commits August 7, 2024 10:24
Co-authored-by: Oskar Stark <oskarstark@googlemail.com>
Co-authored-by: Oskar Stark <oskarstark@googlemail.com>
@Vitexus
Copy link
Contributor Author

Vitexus commented Aug 7, 2024

Ha! I finally found https://github.com/slunak/pushover-php/blob/master/CONTRIBUTING.md :D

I will follow it for further merge requests to make things smooth.

@slunak
Copy link
Owner

slunak commented Aug 7, 2024

Thanks @OskarStark and @Vitexus , I will merge and do a release later today or tomorrow.

@slunak slunak changed the title #26 - Create new group with the Example and Test Create new group with the Example and Test Aug 8, 2024
@slunak slunak merged commit ce70bbd into slunak:master Aug 8, 2024
6 checks passed
@Vitexus Vitexus deleted the create_group branch August 8, 2024 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement Creating a Group
3 participants