-
Notifications
You must be signed in to change notification settings - Fork 2
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
Shortcode Conversations #2
Comments
What we're going to implement on this issue is the back end API call to check for the short code and get it + a front end check short code button It will then put the shortcode in the phone numbers table After that, we will need to implement some switching |
Hey @mabrownnyu and @KarimAl-Rashdan, it sounds like there are two tickets here, and I just want to document that correctly. for MVP, we need an API call from Spoke that goes to Twilio to check if they have access to shortcode. The second phase is to allow a Spoke Administrator to make the above API call. The separate ticket is to allow for switching between a short-code and a phone number. If that sounds good, I can create another issue and put it in our backlog. |
Here's the ticket for switching. StateVoicesNational#5 |
Recovery Moment UI Moment From Spoke for Administrators:
|
|
got a function going in the twilio index file called getShortCode -- need to test it to make sure it's actually processing the number correctly also need to take a look at what the payload looks like if the account has no shortcode |
also github won't let me open another branch? @Arique1104 |
Tomorrow:
|
What's new
Priorities for tomorrow:
|
I think I've figured out the route of things: containers/AdminPhoneNumberInventory.js |
Today:
|
We needed to add the getShortCodes function to the api/schema.js file |
breadcrumbs currently things are mostly connected but there is an error happening in |
Recap of what we learned:
|
Update: Pain points: I need to update the getShortCode function in src/extensions/service-vendors/twilio/index.js at line 779 to have a conditional where if shortcodes are nonexistent I can render a message to let the user know on the frontend. I am receiving a lot of errors in my browser console when I click the check short codes button; my guess is I probably don't have the right permissions. @mabrownnyu Can we sit down and figure out this getShortCode function? |
|
After much deliberation this engineering team has decided that the Toll-Free and the Short-code are similar but seperate issue tickets. Updating this one to reflect short-code, which is complete - pending QA. There will be a linked issue for the creation of Toll-Free. |
Resource created by Karim and Megan assignees to this ticket.
Miro Board Resource
Karim
Megan
The text was updated successfully, but these errors were encountered: