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

Try Sufia Team's Potential Patch for GA Stats Issue #124

Closed
davidschober opened this issue Feb 17, 2017 · 10 comments
Closed

Try Sufia Team's Potential Patch for GA Stats Issue #124

davidschober opened this issue Feb 17, 2017 · 10 comments

Comments

@davidschober
Copy link
Contributor

davidschober commented Feb 17, 2017

DONE

A cron job is added for user stats if necessary.

@carrickr
Copy link

carrickr commented Mar 6, 2017

Try the sufia fix: samvera-deprecated/sufia#3090

@carrickr
Copy link

carrickr commented Mar 6, 2017

@davidschober How sensitive are current GA stats on the production server? There is a known bug where this only works if your rails_env is production. So we either need to completely change the rails_env on nufiaweb-s or do this on nufiaweb-p.

@carrickr carrickr changed the title Add cron job for GA stats if necessary Try Sufia Team's Potential Patch for GA Stats Issue Mar 6, 2017
@davidschober
Copy link
Contributor Author

@carrickr I'm not sure I know what you mean by how sensitive? I don't think we should rebuild nufiaweb-s.

@davidschober
Copy link
Contributor Author

do we think this got sorted in 7.3 @carrickr ?

@davidschober
Copy link
Contributor Author

check after 7.3 #160

@johndorr
Copy link
Collaborator

johndorr commented May 1, 2017

Still not working after upgrade to 7.3

@johndorr johndorr added this to the Open Access Week 2017 milestone May 1, 2017
@chrisdaaz chrisdaaz removed this from the Open Access Week 2017 milestone Nov 28, 2017
@davidschober
Copy link
Contributor Author

@chrisdaaz is this to be abandoned? ? or rewritten.

@chrisdaaz chrisdaaz added this to the Hyrax 2.0 Upgrade milestone Dec 13, 2017
@chrisdaaz
Copy link

@davidschober if i understand this correctly, this should probably work after the hyrax 2.0 update? i'd like to check after then.

@davidschober
Copy link
Contributor Author

davidschober commented Dec 13, 2017 via email

@chrisdaaz chrisdaaz removed this from the Hyrax 2.x Upgrade milestone Sep 18, 2018
@chrisdaaz
Copy link

abandoned -- not sure how to test this on staging, not sure it's worth the effort to fix at this point

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants