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.

Sign up for GitHub

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

Jump to bottom

Update Analytics Report doc with sidebar UI #659

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update Analytics Report doc with sidebar UI #659

wants to merge 1 commit into from

Conversation

jonambas
Copy link
Contributor

@jonambas jonambas commented Mar 1, 2022 •
edited
Loading

What Changed

  • Updates the Analytics Report doc with the new sidebar UI. Updates only some of the images.

How To Test or Verify

PR Checklist

All PRs Checklist

  • Give your pull request a meaningful name.
  • Use lowercase filenames.
  • Apply at least one team label according to which team is the content expert (ie. team-FE or team-SAZ)
  • Pull request approval from the FE team or content experts (see label applied above) that isn't the content creator.

Content Changes Checklist

  • Check that your article looks correct in the preview here or in a Netlify deploy preview.
  • Check the links in your article.
  • Check the images in your article (if there are any)
  • Check to make sure you are using markdown appropriately as outlined in examples/article.md in the root of the project directory and on the momentum doc's preface article
  • Check to make sure the Copy and Tone Guidelines are followed.

Development Changes Checklist (some checks are automatic github actions and will not be listed here. ie. "all tests pass")

  • The appropriate tests are created in cypress/ directory in the root of the project
  • The lighthouse score is passing according to the FE Support Docs' Service Outline SLI/SLOs

@netlify
Copy link

netlify bot commented Mar 1, 2022 •
edited
Loading

✔️ Deploy Preview for support-docs ready!

🔨 Explore the source changes: 06f1b47

🔍 Inspect the deploy log: https://app.netlify.com/sites/support-docs/deploys/621e2fb970f13e00072fe75d

😎 Browse the preview: https://deploy-preview-659--support-docs.netlify.app

@jonambas jonambas added team - UX User Experience team is the subject matter expert and need to approve this PR and removed 👀 needs review type: content change labels Mar 1, 2022
@jonambas jonambas self-assigned this Mar 15, 2022
haleysolomon
haleysolomon approved these changes Mar 15, 2022
View reviewed changes
content/docs/reporting/analytics-report.md

![Finding Saved Reports](media/analytics-report/reportselection.gif)

#### Favorite Reports

For quick and easy access to your reports, you can save them as favorites by clicking the dropdown menu next to the reports in the sidebar.
Copy link
Contributor

Choose a reason for hiding this comment

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

May want to add "You can save both preset reports and reports you create as favorites." Also - is there a limit on favorites? If so, may want to add number of reports you can favorite. If there is no limit, I'm ok with just not mentioning a limit (leave the fact that it's unlimited undocumented) in case we add a limit later.

content/docs/reporting/analytics-report.md
@@ -86,16 +86,22 @@ Save your report for quick access in the future and to avoid reconfiguring your

#### Preset Reports

Your account comes with several preset saved reports which are available from the report dropdown. All preset reports are also available through a menu by clicking the “View All Reports” button in the upper right-hand corner.
Your account comes with several preset saved reports which are available from the sidebar. All preset reports are also available through a menu by clicking the All Reports” button at the bottom of the sidebar.
Copy link
Contributor

Choose a reason for hiding this comment

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

consider adding that you can use a preset report as-is or use it as a starting point for a custom report which you can then save/edit. that might be obvious, I'm not sure 🤷

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@haleysolomon haleysolomon haleysolomon approved these changes

At least 1 approving review is required to merge this pull request.

Assignees

@jonambas jonambas

Labels
:shipit: approved team - UX User Experience team is the subject matter expert and need to approve this PR type: content change
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jonambas @haleysolomon