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

[IMP] Add OVH SMS API User #16

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

Conversation

cvinh
Copy link

@cvinh cvinh commented Jun 27, 2020

Hello Florent
I improved your module in order to take into account the API User in OVH SMS
Best regards

@cvinh
Copy link
Author

cvinh commented Jun 27, 2020

cc @LuceEasyPme

@cvinh cvinh force-pushed the 12.0-add-ovh_api_user branch from b61673b to cbc03bd Compare June 27, 2020 00:17
@fmdl
Copy link
Owner

fmdl commented Jun 27, 2020

Thanks for your PR

Can you modifiy you PR to use this API without user ?

regards

@cvinh
Copy link
Author

cvinh commented Jun 29, 2020

Can you modifiy you PR to use this API without user ?

Done

@cvinh cvinh force-pushed the 12.0-add-ovh_api_user branch from c59b42b to 27c5b85 Compare July 1, 2020 17:56
@cvinh
Copy link
Author

cvinh commented Jul 1, 2020

Sorry I forgot to rebase

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

Successfully merging this pull request may close these issues.

2 participants