utopia-php / messaging

Lite & fast micro PHP messaging library that is **easy to use**.
MIT License
14 stars 64 forks source link

Feat 6404 smtp2 go messaging adapter #51

Open Ayaan49 opened 8 months ago

Ayaan49 commented 8 months ago

What does this PR do?

Added SMTP2GO messaging adapter to improve Appwrite messaging.

Test Plan

I created a free SMTP2GO plan, developed the adapter, and tested it by creating a test file inside the "tests" directory. I added the SMTP2GO credentials through environment variables in the Docker Compose file to connect with my SMTP2GO account and verify if emails are being sent. It sent emails successfully. I ran the test case, and it executed without any issues. The command used to run the test case is docker exec -it messaging-tests-1 ./vendor/bin/phpunit tests/e2e/Email/Smtp2goTest.php. All testing was conducted within Docker containers.

Related PRs and Issues

appwrite/appwrite#6404

Have you read the Contributing Guidelines on issues?

Yes

Ayaan49 commented 8 months ago

Fixed the linting errors @gewenyu99

gewenyu99 commented 8 months ago

Hey,

Due to time constraints, I'm going to mark this PR hacktoberfest-accepted for now so you get DO's Hacktoberfest rewards. We'll continue to work with you on this issue for review and merge.

When it is merged, we'll contact you for Appwrite-specific Hacktoberfest swag.

Thanks for helping us improve Appwrite!

Ayaan49 commented 8 months ago

@stnguyen90 @gewenyu99 My tests are succesfull using docker containers. Here is the command docker exec -it messaging-tests-1 ./vendor/bin/phpunit tests/e2e/Email/Smtp2goTest.php smtp2go pass appwrite smtp2go console appwrite

tessamero commented 7 months ago

Hello @Ayaan49

Thank you for your contribution to Hacktoberfest 2023! We've noticed that your PR is still pending and requires some updates based on our engineering team's feedback.

We would love to see your PR successfully merged and send you the Appwrite swag as a token of appreciation. To remain eligible for the swag, please address the pending suggestions and/or ensure the tests pass by Friday, November 17th. If the PR isn't updated by then, we will unfortunately have to close it due to the end of the Hacktoberfest event.

Looking forward to your updates and thank you!

Ayaan49 commented 7 months ago

Hello @tessamero , I have already addressed the changes and mentioned the maintainers but I didn't get any reply. Also my tests are passing inside the Docker containers and I have provided ss of the same. Please look into it and tell me if it's ok or not.

gewenyu99 commented 2 months ago

Hey there! There were a lot of big PRs during this Hacktoberfest, and we wanted to give everyone ample time to collaborate with our engineering team. If you were able to merge your PRs during October, amazing. If it’s still not merged, don’t worry about it either. Either way, we’ve got your Hacktoberfest swag minted and ready to ship.

Please comment with your Discord username here so we can contact you about your shipping information to deliver your Hacktoberfest swag.

Ayaan49 commented 2 months ago

@gewenyu99 ayaan49

gewenyu99 commented 2 months ago

Will try to reach out soon, compiling everyone's names (there's a lot!). Appreciate your patience.