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

WIP: Update invitation email wording #716

Merged
merged 3 commits into from
Aug 2, 2016
Merged

WIP: Update invitation email wording #716

merged 3 commits into from
Aug 2, 2016

Conversation

jml
Copy link

@jml jml commented Aug 2, 2016

Fixes #692, fixes #653.

Current wording

You've been invited to join the organization ":org_id" on Weave Cloud.

Click this link to login to Weave Cloud:

Log in to Weave Cloud

This single-use login link is valid for 3 days. You can generate a new one at any time by visiting Weave Cloud

You need to run probes on your Docker hosts to send information to Weave Cloud. Instructions for that are available once you log in.

Thanks, the Weaveworks team

<%body%>

Proposed wording

First time

:user_name has invited you to access the ":org_name" monitoring instance on Weave Cloud.

Click this link to login to Weave Cloud:

Log in to Weave Cloud

This single-use login link is valid for 3 days. You can generate a new one at any time by visiting Weave Cloud

You need to run probes on your Docker hosts to send information to Weave Cloud. Instructions for that are available once you log in.

Thanks, the Weaveworks team

<%body%>

Differences

  • org_name not org_id
  • include user name
  • "monitoring instance" not "organization"

Already have an account

:user_name has granted you access to the ":org_name" monitoring instance on Weave Cloud.

View :org_name

Thanks, the Weaveworks team

<%body%>

Differences:

  • No mention of "logging in", assume they have that down
  • No mention of Docker probes, because the instance already exists
  • Mention user_name
  • Use org_name, not org_id
  • Link directly to the specific organization

davkal added 3 commits August 1, 2016 17:09
* PrivatePage wraps components that are only accessible when logged in
* PrivatePage contains the toolbar
* It does a cookie-check and redirects to login when no cookie

Fixes #702
Fixes #679
Adds PrivatePage container to wrap internal components
@jml jml merged commit 8e870c7 into 692-invitation-wording Aug 2, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants