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

HYC-1939 - Change work stats to pulling from local database #1128

Merged
merged 12 commits into from
Nov 14, 2024
Merged

Conversation

bbpennel
Copy link
Member

@bbpennel bbpennel commented Nov 6, 2024

https://unclibrary.atlassian.net/browse/HYC-1939

  • Public work analytics page now pulls download stats from local database
    • Now only pulls 12 months of data, to match the view
  • Disabled the public file analytics page, since it was broken
  • Updated admin dashboard work analytics page to use local database for download stats
    • matomo.rb now redirects download related events to the local database (for methods that we have known uses of, I did not update all methods)
    • dashboard pulls monthly stats instead of daily
    • refactor of dashboard stat combination to use hashes for look ups, to greatly improve performance
    • Benchmarking added to help pinpoint which parts of the data load are slow. We might want to remove this later, but it'd be helpful to see in prod.
  • Admin dashboard now shows all the buttons to get to alternative reports, which were hidden by css issues before
  • Bug fix for issue cause fileset_id to be recorded as work id in hyc_download_stats

@bbpennel bbpennel changed the title WIP: HYC-1939 - Change work stats to pulling from local database HYC-1939 - Change work stats to pulling from local database Nov 14, 2024
@davidcam-src davidcam-src merged commit 2c78945 into main Nov 14, 2024
4 checks passed
@davidcam-src davidcam-src deleted the hyc-1939 branch November 14, 2024 19: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.

2 participants