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
7079 guides download all and analytics #7110
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. I didn't test the analytics code but it's trying to do makes sense.
@@ -117,9 +119,6 @@ Ingested files can be downloaded in several different ways. | |||
|
|||
- The variable metadata for the file in DDI format | |||
|
|||
- A subset of the columns of the data |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, I assume we forgot to remove this line about subset as part of #6098.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense to me.
@mheppler Quick question, should https://raw.githubusercontent.com/IQSS/dataverse/2ac69d7503d12a1013530683db4dbcbb9b8bb969/doc/sphinx-guides/source/_static/installation/files/var/www/dataverse/branding/analytics-code.html be downloadable from the guides? I see analytics-code.html highlighted but not downloadable. |
What this PR does / why we need it:
Which issue(s) this PR closes:
Closes #7079 Access Dataset + Download "All Files" - Update Guides and Analytics
Special notes for your reviewer:
N/A
Suggestions on how to test this:
Need to configure analytics, using new sample code provided in Installation Guide, should see "Original Format ZIP" and "Archival Format (.tab) ZIP" recorded in the analytics reports.
Does this PR introduce a user interface change? If mockups are available, please link/include them here:
N/A
Is there a release notes update needed for this change?:
Need release notes notifying sysadmins to update their analytics code if configured already, so the file size "(5.1 MB)" is not included in the Event Category recorded in the Google Analytics reports.
Additional documentation:
N/A