Updates to Bandwidth Integration HOW-TO based on my experience#2205
Updates to Bandwidth Integration HOW-TO based on my experience#2205rhayhow wants to merge 4 commits intoProgressiveCoders:mainfrom rhayhow:main
Conversation
Update HOWTO_INTEGRATE_BANDWIDTH.md for first feedback cycle.
Update HOWTO_INTEGRATE_BANDWIDTH.md
|
I have shared some of the details of this pull request with @Arique1104 so they should have a sense of what the changes are. |
Arique1104
left a comment
Arique1104
left a comment
There was a problem hiding this comment.
That's my only addition, to add on some documentation to our Service Managers page.
| # Bandwidth Integration | ||
|
|
||
| Bandwidth.com is a telephone service API company. To use Bandwidth, set `DEFAULT_SERVICE=bandwidth`. The `sticky-sender` and `num-picker` service managers are required for the Bandwidth extension to work. `sticky-sender` must come before `numpicker-basic` in the `SERVICE_MANAGERS` environment variable. | ||
| Bandwidth.com is a telephone service API company. To use Bandwidth, set `DEFAULT_SERVICE=bandwidth`. The `sticky-sender` and `numpicker-basic` service managers are required for the Bandwidth extension to work. `sticky-sender` must come before `numpicker-basic` in the `SERVICE_MANAGERS` environment variable. NOTE: Because service managers are complicated, there is not an entry for SERVICE_MANAGERS in the Spoke Documentation Hub. Just add it as an enviromental variable. |
There was a problem hiding this comment.
Here's the documentation for Service Managers
We could have the Service Managers doc (linked above) can have a new section for ngpvan and beneath it, link the HOWTO Integrate Bandwidth. I'd be happy to make those changes and push them to this branch. :)
There was a problem hiding this comment.
I will look at what is required to update the Service Managers doc. Please feel free to close the request :)
|
@rhayhow, thank you for these changes! You should see them reflected in the Bandwidth Community Input Pull Request I'd love it if you could review those changes and give any feedback. I had a lot of trouble getting your branch pulled to my local, so I ended up incorporating your edits and making a couple of markdown readability edits and I hope the final product is useful. With your permission, I'd like to close this pull request. |
|
Please feel free to close the pull request. |
2 participants
Fixes # (issue)
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change, and any blockers that make your change a WIP
Checklist: