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

Improve Appwrite Messaging with Socketlabs Adapter #62

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

drraghavendra
Copy link

@drraghavendra drraghavendra commented Oct 30, 2023

Description

The Socketlabs adapter for Appwrite Messaging can be improved by adding support for more features, such as tracking email opens and clicks, managing email templates, and supporting multiple email providers. The adapter can also be made more efficient, user-friendly, and customizable. Additionally, the adapter could be improved by providing a unified API and adding support for batch processing and retries.

The Salient features are listed below

  1. Add more features: Tracking email opens and clicks, managing email templates, and supporting multiple email providers.
  2. Improve efficiency: Send emails more quickly and efficiently.
  3. Improve user-friendliness: Provide better documentation and tutorials.
  4. Improve customizability: Make the adapter more customizable and extensible.
  5. Provide a unified API: Make it easier for Appwrite developers to use all of the features of the adapter.
  6. Support batch processing: Send multiple emails at once.
  7. Support retries: Ensure that all emails are eventually delivered.

What does this PR do?

appwrite/appwrite#6405

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

@drraghavendra
Copy link
Author

Sir @PineappleIOnic as per your suggestions and inference Pull Request made.

@PineappleIOnic
Copy link
Member

Can you please produce an image of this adapter working?

@gewenyu99
Copy link

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!

@gewenyu99
Copy link

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.

@drraghavendra
Copy link
Author

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.

Respected @gewenyu99 my discord username raghavendra99

@gewenyu99
Copy link

Hey, adding to a spread sheet as we speak. Will be reaching out ASAP, thanks for your patience!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants