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 46elks #6861

Closed
4 tasks
Haimantika opened this issue Oct 9, 2023 · 11 comments · May be fixed by utopia-php/messaging#54
Closed
4 tasks

💬 Improve Appwrite Messaging with 46elks #6861

Haimantika opened this issue Oct 9, 2023 · 11 comments · May be fixed by utopia-php/messaging#54
Assignees
Labels
hacktoberfest Issues that can win you some cool swags!

Comments

@Haimantika
Copy link
Contributor

Haimantika commented Oct 9, 2023

💭 Introduction

Having an app is cool, but having integrations is even cooler! 😎 It always brings a smile to my face when a website has the option to sign in with my GitHub account, so I don't have to go over the lengthy process of creating an account… The same can be said about messages! Every app with the option to enable push notifications, email, or SMS notifications gives the users confidence they won't miss anything important. We all have our channels we check the most, and having the ability to connect some app directly in there is always a positive.

Appwrite will use Utopia PHP messaging library to allow developers to get in touch with users using one of its many messaging adapters, such as Email, Discord, or Slack.

Since the messaging library supports adapters, there is always space for more. 😎 Each provider implements a method to send the message to the correct channel (usually using an HTTP request or a library) alongside a few configuration variables and methods to set what the adapter allows.

Your task is to implement support for 46elks in the Utopia PHP messaging library. This goes under SMS category, so feel free to look at existing ones in the adapters folder. We have prepared detailed documentation on how to add a new messaging adapter. Please read the document and ensure you understand them before working on this issue.

🎯 Requirements

✅ Task Summary

  • Ask to be assigned to the issue.
  • Wait to be assigned.
  • Implement Utopia PHP logger adapter for SMS/46elks.
  • Submit a pull request in utopia-php/messaging.

If you have questions, need any help, or just want to hang out, make sure to join us on our Discord server.

Happy Appwriting!

@DhairyaMajmudar
Copy link

@Haimantika I would like to work on this. Can you assign it to me?

@Haimantika Haimantika added the hacktoberfest Issues that can win you some cool swags! label Oct 9, 2023
@Haimantika
Copy link
Contributor Author

@Haimantika I would like to work on this. Can you assign it to me?

Hi @DhairyaMajmudar have assigned the issue to you! Thank you for showing interest in contributing to Appwrite! Happy Hacktoberfest 🎃

Notes:

Please update us with your progress every 3 days, so that we know that you are working on it.
Join us on Discord - https://appwrite.io/discord to chat about Hacktoberfest and Appwrite!

@IftequarIffu
Copy link

@Haimantika Would like to work on this if it is unsolved

@DhairyaMajmudar
Copy link

@Haimantika Would like to work on this if it is unsolved

Sorry But I am working on this issue pls. check another one

@Haimantika
Copy link
Contributor Author

@Haimantika Would like to work on this if it is unsolved

Sorry But I am working on this issue pls. check another one

Are you still working on it? I remember you mentioned that it is not supported in India

@DhairyaMajmudar
Copy link

@Haimantika Would like to work on this if it is unsolved

Sorry But I am working on this issue pls. check another one

Are you still working on it? I remember you mentioned that it is not supported in India

Yes ma'am I have cloned the repo in my local system and also wrote the structural code as given in the docs. provided along with that I also emailed the 46elks support team for a dummy account. and below is their reply. Please suggest what requirement should I reply them back

image

@Haimantika
Copy link
Contributor Author

@Haimantika Would like to work on this if it is unsolved

Sorry But I am working on this issue pls. check another one

Are you still working on it? I remember you mentioned that it is not supported in India

Yes ma'am I have cloned the repo in my local system and also wrote the structural code as given in the docs. provided along with that I also emailed the 46elks support team for a dummy account. and below is their reply. Please suggest what requirement should I reply them back

image

I think they are asking for your personal details. Send them accordingly

@DhairyaMajmudar
Copy link

@Haimantika ma'am I have created a PR utopia-php/messaging#54 pls. review it.

@DhairyaMajmudar
Copy link

@Haimantika ma'am my PR has been reviewed by @wess pls. consider merging it in the codebase 🙏

@Haimantika
Copy link
Contributor Author

@Haimantika ma'am my PR has been reviewed by @wess pls. consider merging it in the codebase 🙏

Your PR is reviewed and has the hacktoberfest-accepted label, which means it counts valid for Hacktoberfest. Merging will take some time :)

@adityaoberai
Copy link
Member

Closing this issue. Thank you all so much for participating in Hacktoberfest 2023! We can't wait to welcome you all during HF 2024! Stay tuned for a lot more amazing issues from the Appwrite team!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest Issues that can win you some cool swags!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants