-
Notifications
You must be signed in to change notification settings - Fork 3
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
Integrate the GA with API keys #2477
Conversation
Have you looked at the existing keys here? https://github.com/osulp/Scholars-Archive/blob/main/config/initializers/hyrax.rb#L68 A Pull Request last year tried to remove the analytics.yml file here, since it wasn't being used and was confusing, but the file was expected to be there by another file. #2370 |
@wickr I have gotten a chance to look at it, and Corey and I also sit down and talk about it. We look through |
Ok, I see what you're saying. But if this file was never actually used, but the system of retrieving analytics data worked up until a bit ago, I would think it's something else. |
I will take a look and see where it will go from there but thanks for letting me know about it Ryan. |
I'm not convinced it was working until a little bit ago. @carakey @simholt any idea how long analytics have been out of sync? As far as I can tell, the stats were synced from dspace ~6 years ago, then stopped updating around Jan 2022 for some reason. From what I can trace, Hyrax attempts to pull analytics from GA, failing that they use |
I'm guessing we lost another config or an API key ( Or, there was a Rake task running for stats, but it didn't get included in one of the system migrations. There might be helpful info here: uclibs/scholar_uc_legacy#1767 |
@CGillen No idea, they have been problematic off and on since the move to Hyrax. |
@CGillen All I can say for sure is that it was reported as busted by an OSU faculty on March 13. |
Add in the
Google Analytics
key to the YML file to see if this fix theScholars Archive
analytics.#2442