Skip to content
This repository has been archived by the owner on Jul 1, 2023. It is now read-only.

Interaction with new send SMS permission #5

Closed
michaelmcandrew opened this issue Jan 26, 2018 · 3 comments
Closed

Interaction with new send SMS permission #5

michaelmcandrew opened this issue Jan 26, 2018 · 3 comments

Comments

@michaelmcandrew
Copy link

Hey there,

I am not sure if / how this API might be affected by the new Send SMS permission that is (hopefully!) coming to core but wanted to create an issue here to flag up that it is on its way, at least: civicrm/civicrm-core#11590

@jaapjansma
Copy link
Contributor

I dont know either. I do think this extension works without that permission and it could be that it is desired case. Imagine a use case where we use this api in civirules and a rule is setup that after a new donation comes in a thank you sms should be send. Than the user and the permission used are the one of the anonymous user.

@michaelmcandrew
Copy link
Author

That makes two of us then :) Something to be aware of at least.

@kainuk
Copy link
Contributor

kainuk commented Jul 1, 2023

Added it as a known issue to the README, here https://lab.civicrm.org/extensions/smsapi/-/blob/main/README.md .

@kainuk kainuk closed this as completed Jul 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants