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

Add a tour screen once a broadcast is approved #3632

Merged
merged 2 commits into from
Sep 23, 2020

Conversation

quis
Copy link
Member

@quis quis commented Sep 17, 2020

For a training broadcast the user doesn’t get that immediate feedback that something has happened, like they would with a real alert, or even sending themselves a text message.

This commit adds another tour-style page which will interrupt their journey and hopefully reinforce the message we’ve given them earlier in the tour.

We’re adding this because we’ve found in research that users don’t have a good grasp of the consequences and severity of emergency alerts, versus regular text messages.

Before After
image image
image image

@quis quis force-pushed the training-broadcast-approved branch 2 times, most recently from b4b286c to e4d694a Compare September 17, 2020 12:50
For a training broadcast the user doesn’t get that immediate feedback
that something has happened, like they would with a real alert, or even
sending themselves a text message.

This commit adds another tour-style page which will interrupt their
journey and hopefully reinforce the message we’ve given them earlier in
the tour.

We’re adding this because we’ve found in research that users don’t have
a good grasp of the consequences and severity of emergency alerts,
versus regular text messages.
@quis quis force-pushed the training-broadcast-approved branch from e4d694a to 5c22bd5 Compare September 21, 2020 08:41
@quis
Copy link
Member Author

quis commented Sep 21, 2020

@karlchillmaid I’ve swapped the order of the first paragraph so it joins into one sentence, and leads with ‘Notify has not broadcast your alert’ which is the response to what the user has just done. I don’t think we need to say ‘still’ because the idea is users will swap back and forth between training and real depending on the situation.

If you’re happy with this now then I’m happy with this too.

image

I’ve swapped the order of the first paragraph so it joins into one
sentence, and leads with ‘Notify has not broadcast your alert’ which
is the response to what the user has just done. I don’t think we need
to say ‘still’ because the idea is users will swap back and forth
between training and real depending on the situation.
@quis quis marked this pull request as ready for review September 21, 2020 10:42
@karlchillmaid
Copy link
Contributor

@quis Yeah, definitely an improvement.

</p>
<p class="govuk-body heading-medium">
<a class="govuk-link govuk-link--no-visited-state" href='{{ url_for(".service_dashboard", service_id=current_service.id) }}'>
Continue to dashboard
Copy link
Contributor

Choose a reason for hiding this comment

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

Surprised me that you'd take them to the dashboard, I would have assumed that we would have sent them to view the broadcast they sent so they learn what they should expect to see in the real case.

Copy link
Member Author

Choose a reason for hiding this comment

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

I think once we make the dashboard better it will be the right place to take them. Appreciate it’s different from the ‘live‘ journey at the moment.

@quis quis merged commit dcd48f9 into master Sep 23, 2020
@quis quis deleted the training-broadcast-approved branch September 23, 2020 10:07
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.

None yet

3 participants