Skip to content

Conversation

@Sukriti-m
Copy link

What does this PR do?

This PR implements BulkSMS messaging adapter

Test Plan

  1. Create Your BulkSMS Account

  2. Sign up for an account on BulkSMS.

  3. Set up Sender Information

  4. Set up your phone number

  5. Send the message.

Screenshot 2023-10-09 235907
Image shows the messages sent from a user's account

Related PRs and Issues

Close #6385

Have you read the Contributing Guidelines on issues?

Yes

@wess wess self-assigned this Oct 19, 2023
@wess
Copy link
Contributor

wess commented Oct 19, 2023

Please, even if commented out, write a test, and be sure to run pint/linter. Thank you!

@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!

@Sukriti-m
Copy link
Author

g 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!

Thank You. I will be soon creating a PR with the required changes.

Copy link
Contributor

@wess wess left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Corrected.

@gewenyu99
Copy link

#48

@Sukriti-m Sukriti-m closed this by deleting the head repository Aug 23, 2024
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.

💬 Improve Appwrite Messaging with BulkSMS Adapter

3 participants