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

741 default instance-wide optout message #882

Merged
merged 5 commits into from
Sep 17, 2018

Conversation

mathemagica
Copy link
Contributor

Added a new place to edit org-specific optout message in org settings, and if set, it'll override both the env var instance-wide setting OPT_OUT_MESSAGE and also the hard coded default "I'm opting you out of texts immediately. Have a great day." Test this by going into your name (in top right) -> your org -> settings and then editing the "Default opt-out message" setting. #741

@@ -1,5 +1,6 @@
import PropTypes from 'prop-types'
import React from 'react'
import TextField from 'material-ui/TextField'
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we remove this unused import?


<Form.Field
label='Default Opt-Out Message'
name='optOutMessage'
Copy link
Contributor

Choose a reason for hiding this comment

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

can we add the prop fullWidth here so the user can see the full message (versus having to scroll within the form field) ?

Copy link
Contributor

@shakalee14 shakalee14 left a comment

Choose a reason for hiding this comment

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

Thanks! Some changes requested - specifically adding a small improvement to the UI experience

@shakalee14 shakalee14 added the S-ready for stage-main (qa) Status (ADMINS ONLY): PR label for those ready to be added for stage: Approved, tests, etc label Sep 12, 2018
@shakalee14 shakalee14 mentioned this pull request Sep 12, 2018
@shakalee14 shakalee14 merged commit 653ecf0 into main Sep 17, 2018
@schuyler1d schuyler1d deleted the 741-default-instancewide-optout-message branch March 26, 2019 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-ready for stage-main (qa) Status (ADMINS ONLY): PR label for those ready to be added for stage: Approved, tests, etc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants