-
Notifications
You must be signed in to change notification settings - Fork 329
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
feat(templates): add Opt In Builder template #261
base: main
Are you sure you want to change the base?
Conversation
Closes twilio-labs#1: More flexible message frequency configuration
@myfancypants Hey there! Is there a process for me to request a review on this so that I can merge and add it to the CodeExchange? There's a bunch of functionality in this template, so I recorded a video demo of what it does here: https://www.loom.com/share/c931fc90f745403795153bb171b1e1a1 The intent is to help out all of our messaging customers that now need to capture opt-in for long codes under A2P 10DLC. |
@@ -14,15 +14,15 @@ module.exports = { | |||
ecmaVersion: 2018, | |||
}, | |||
rules: { | |||
'prettier/prettier': 'allow', | |||
'import/no-extraneous-dependencies': 'allow', | |||
'prettier/prettier': 1, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was receiving errors from prettier saying that allow
was deprecated, so I updated these configurations to match their new convention.
Description
Creates an embeddable, compliant opt-in form that can be used for short code applications and A2P 10DLC. Key features:
See README for setup details.
Checklist
npm test
locally and it passed without errors.Related issues