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

Update chart copy #1103

Merged
merged 3 commits into from
Jan 17, 2024
Merged

Update chart copy #1103

merged 3 commits into from
Jan 17, 2024

Conversation

nick-mon1
Copy link
Contributor

@nick-mon1 nick-mon1 commented Jan 16, 2024

Summary

Update copy to read 90 days for reporting where applicable.

Impacted Areas of the Site

data downloads
Screen Shot 2024-01-16 at 9 27 06 AM

sessions
Screen Shot 2024-01-16 at 9 28 32 AM

top pages
@smarina04 Still keep this at 30 days?
Screen Shot 2024-01-16 at 9 41 17 AM

current visitors and past 30 days for sessions
homepage


This pull request is ready to merge when...

  • Tests have been updated (and all tests are passing)
  • This code has been reviewed by someone other than the original author
  • The change has been documented

@nick-mon1 nick-mon1 changed the title Updated 30 days to 90 in charts Update 30 days to 90 in charts Jan 16, 2024
@nick-mon1 nick-mon1 requested a review from smarina04 January 16, 2024 14:34
@nick-mon1 nick-mon1 self-assigned this Jan 16, 2024
@nick-mon1 nick-mon1 marked this pull request as ready for review January 16, 2024 14:35
@nick-mon1 nick-mon1 changed the title Update 30 days to 90 in charts Update chart copy Jan 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants