Skip to content

admin: allow Twilio messaging service SID as "from number"#1899

Merged
mastercactapus merged 37 commits into
masterfrom
admin-toolbox-messaging-service-support
Sep 23, 2021
Merged

admin: allow Twilio messaging service SID as "from number"#1899
mastercactapus merged 37 commits into
masterfrom
admin-toolbox-messaging-service-support

Conversation

@dctalbot
Copy link
Copy Markdown
Contributor

@dctalbot dctalbot commented Sep 14, 2021

  • Identified the issue which this PR solves.
  • Read the CONTRIBUTING document.
  • Code builds clean without any errors or warnings.
  • Added appropriate tests for any new functionality.
  • All new and existing tests passed.
  • Added comments in the code, where necessary.
  • Ran make check to catch common errors. Fixed any that came up.

Description:
This PR adds support for using Twilio's messaging service SID to send messages.

@dctalbot dctalbot marked this pull request as ready for review September 15, 2021 16:46
@m17ch m17ch changed the title admin: allow messenger SID as "from number" admin: allow Twilio messaging service SID as "from number" Sep 15, 2021
Comment thread config/config.go Outdated
Copy link
Copy Markdown
Contributor

@m17ch m17ch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm 👍

@mastercactapus mastercactapus merged commit eaa88fb into master Sep 23, 2021
@mastercactapus mastercactapus deleted the admin-toolbox-messaging-service-support branch September 23, 2021 15:22
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.

4 participants