Skip to content
This repository has been archived by the owner on May 19, 2020. It is now read-only.

Update notification copy #1134

Merged
merged 7 commits into from
Jun 26, 2017
Merged

Update notification copy #1134

merged 7 commits into from
Jun 26, 2017

Conversation

rememberlenny
Copy link
Contributor

No description provided.

@rememberlenny rememberlenny force-pushed the lkb-copy_for_invite_notices branch from ae70e7c to 4d53bb9 Compare June 23, 2017 16:44
@rememberlenny rememberlenny force-pushed the lkb-copy_for_invite_notices branch from a8aedb2 to a4f5756 Compare June 23, 2017 21:25
@rememberlenny
Copy link
Contributor Author

This should go green and is finished @jcscottiii @el-mapache @fureigh

@rememberlenny rememberlenny requested a review from jcscottiii June 26, 2017 15:08
`has been associated to this ${viewTypeNouns[currentViewedType].singular}` +
` and their ${viewTypeNouns[currentViewedType].singular}` +
' roles can be controlled below.';
description = `No verified cloud.gov account was found for ${email}, so ` +
Copy link
Contributor

@jcscottiii jcscottiii Jun 26, 2017

Choose a reason for hiding this comment

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

I'm wondering if the typical user will know what a verified account means. I can see just starting off the description with The user has been sent...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sounds good!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jcscottiii Updated!

Copy link
Contributor

@jcscottiii jcscottiii left a comment

Choose a reason for hiding this comment

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

looks good. just a comment on wording. not too sold on either but would like your thoughts.

@jcscottiii jcscottiii merged commit 4c4006c into master Jun 26, 2017
@jcscottiii jcscottiii deleted the lkb-copy_for_invite_notices branch June 26, 2017 16:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants