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

Clickatell Integration outdated #15

Closed
zazdev123 opened this issue Feb 5, 2023 · 5 comments
Closed

Clickatell Integration outdated #15

zazdev123 opened this issue Feb 5, 2023 · 5 comments

Comments

@zazdev123
Copy link
Contributor

Hello,
I allow myself to open an "issue" because the Clickatell API has changed after 7 years, and the request form has changed.
I'll make a pull request as soon as I fix the problem.
Have a good day.

@zazdev123
Copy link
Contributor Author

Sorry, i don't know how to include 2 files in 1 pull request, I'm new on GitHub.

@luke-
Copy link
Contributor

luke- commented Feb 6, 2023

Thanks for you contribution! Is your change finished? If so, we could release a new version of the module.

@zazdev123
Copy link
Contributor Author

Thanks for you contribution! Is your change finished? If so, we could release a new version of the module.

I'll take a closer look tomorrow! It was a pleasure to contribute to the module!

@zazdev123
Copy link
Contributor Author

I just looked, that's all there was to change. I plan to add suppliers to the module in the future, you can count on me.
Have a good day ! :D

@zazdev123 zazdev123 reopened this Feb 7, 2023
@luke-
Copy link
Contributor

luke- commented Feb 7, 2023

@zAlwaysTheSun Nice to hear! :-) A SMS driver could possibly also be something for the two-factor module.
https://github.com/humhub/twofa

Just released 0.7.6
https://github.com/humhub-contrib/sms/releases/tag/v0.7.6

@luke- luke- closed this as completed Feb 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants