Skip to content

Remove redundant PII warning#1612

Merged
jmhooper merged 1 commit intomasterfrom
jmhooper-remove-redundant-message
Aug 15, 2017
Merged

Remove redundant PII warning#1612
jmhooper merged 1 commit intomasterfrom
jmhooper-remove-redundant-message

Conversation

@jmhooper
Copy link
Contributor

Why: We added a banner that displays the "Do not use real personal
information" message across the entire site in dev and in the lower
envs. That made the warnings on the verification forms redundant.

Copy link
Contributor

@hursey013 hursey013 left a comment

Choose a reason for hiding this comment

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

LGTM

**Why**: We added a banner that displays the "Do not use real personal
information" message across the entire site in dev and in the lower
envs. That made the warnings on the verification forms redundant.
@jmhooper jmhooper force-pushed the jmhooper-remove-redundant-message branch from 4cf103b to 8771844 Compare August 15, 2017 19:44
@jmhooper jmhooper merged commit bb68829 into master Aug 15, 2017
@jmhooper jmhooper deleted the jmhooper-remove-redundant-message branch December 12, 2017 20:15
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.

2 participants