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

Add BRCK #54

Open
wants to merge 1 commit into
base: release/16.0
Choose a base branch
from
Open

Conversation

matalaweb
Copy link

Adding BRCK (brck.com) as a provider.

@billsimon
Copy link
Contributor

Hi @matalaweb , are you planning to implement inbound messaging also or only outbound?

@matalaweb
Copy link
Author

My apologies, I do intend to implement inbound. I will create a new PR when implemented.

@billsimon
Copy link
Contributor

No worries; let's keep using this same branch & PR. Comment when you've got the inbound messaging done and ready for review.

@matalaweb
Copy link
Author

I'm struggling to find the inbound message handling method. Could you provide some guidance as to where I should be looking? I apologize, this is my first time touching a fbpx module.

@billsimon
Copy link
Contributor

@matalaweb Sorry I missed your question last week. Take a look at any of the other provider-*.php files. You implement the callPublic($connector) function such that it receives the webhook sent by the provider (in whatever format they send, usually JSON), use the $connector->getMessage function to ingest it into the SMS system, and if it's a MMS, use the $connector->addMedia function to ingest the media attachments. When done, call $connector->emitSmsInboundUserEvt(...) to signal that a new SMS has been received and respond to the provider with the appropriate code (usually 200 or 202).

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.

2 participants