-
Notifications
You must be signed in to change notification settings - Fork 231
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
'Share user research' guidance #2559
Conversation
✅ You can preview this change here:
To edit notification comments on pull requests, go to your Netlify site configuration. |
@christopherthomasdesign The PR to fix your Markdown code wrapping got merged by the way 👍 |
b5b2f48
to
56b7da1
Compare
Hi @christopherthomasdesign, @claireashworth I've had a go at putting revised content from the doc into this branch, admittedly in semi-haste but feel like this is close to ready. Would you care to review and see if there's anything else I might've missed for us to look at to get published? I've highlighted one section I've added that would appreciate a review. |
Useful findings to share might be things like: | ||
|
||
- summaries of user research you’ve done | ||
- prototypes or ideas that you’ve tested | ||
- analytics from your service | ||
- responses to any calls for information we give in the 'Research on this...' section of our pages |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Adding these examples is new — so appreciate any thoughts on whether this looks accurate.
Sorry @calvin-lau-sig7 due to the way Nunjucks renders before Markdown we need to use HTML for the link: Before{% set callout %}
Share information responsibly. [You must get informed consent from your participants](https://www.gov.uk/service-manual/user-research/getting-users-consent-for-research) before you can share findings of any research they’ve participated in.
{% endset %} After{% set callout %}
Share information responsibly. <a href="https://www.gov.uk/service-manual/user-research/getting-users-consent-for-research">You must get informed consent from your participants</a> before you can share findings of any research they’ve participated in.
{% endset %} Otherwise it looks like Markdown is injecting |
6b5d3ef
to
1a18d46
Compare
1a18d46
to
759d6a1
Compare
Typos First line Spacing Add more examples of research Apply suggestions from code review Coding typo Small code typo
759d6a1
to
d92a666
Compare
@calvin-lau-sig7 There was one more -<p>
{{ govukWarningText({
html: callout,
iconFallbackText: "Warning"
}) }}
-</p> All passing now and deployment is working again |
Draft of some guidance to help teams share useful research findings about things in the Design System