Skip to content

add analytics and privacy policy#2828

Merged
orbeckst merged 2 commits intodevelopfrom
add-privacy-policy
Jul 8, 2020
Merged

add analytics and privacy policy#2828
orbeckst merged 2 commits intodevelopfrom
add-privacy-policy

Conversation

@orbeckst
Copy link
Member

@orbeckst orbeckst commented Jul 8, 2020

Fixes #2813

Changes made in this Pull Request:

  • add GoatCounter for basic analytics
  • add link to our Privacy Policy

PR Checklist

  • Tests?
  • Docs?
  • CHANGELOG updated?
  • Issue raised/referenced?

@pep8speaks
Copy link

Hello @orbeckst! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 254:80: E501 line too long (101 > 79 characters)

Line 835:80: E501 line too long (94 > 79 characters)

@orbeckst orbeckst requested review from jbarnoud and lilyminium July 8, 2020 00:48
@orbeckst orbeckst marked this pull request as draft July 8, 2020 00:50
@orbeckst orbeckst force-pushed the add-privacy-policy branch from 9c8b1db to fd3cbf5 Compare July 8, 2020 00:52
@orbeckst orbeckst marked this pull request as ready for review July 8, 2020 00:53
@orbeckst
Copy link
Member Author

orbeckst commented Jul 8, 2020

This is what the rendered footer looks like:

privacy_footer

Copy link
Member

@lilyminium lilyminium left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @orbeckst for adding this! LGTM, just the thought below.

Copy link
Contributor

@jbarnoud jbarnoud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the CSS file that gets deleted?

@lilyminium
Copy link
Member

I think it was custom CSS for the alabaster theme. I switched our docs to the RTD theme and used CSS from custom.css (https://github.com/MDAnalysis/mdanalysis/pull/2399/files#diff-806739c80780d4a61187355682fd6640L9) but never deleted the other file in #2399.

@orbeckst
Copy link
Member Author

orbeckst commented Jul 8, 2020

What is the CSS file that gets deleted?

readable.css was used previously and imported in custom.css. We are not using readable.css anymore (it should have been deleted when we switched over to the MDA-RDT User Guide theme).

EDIT: sorry for the noise – somehow I didn't have @lilyminium 's reply on my page when I saw the question (stale page in browser...)

@orbeckst
Copy link
Member Author

orbeckst commented Jul 8, 2020

See also PR MDAnalysis/pmda#129 for such a removal.

@codecov
Copy link

codecov bot commented Jul 8, 2020

Codecov Report

Merging #2828 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #2828   +/-   ##
========================================
  Coverage    92.22%   92.22%           
========================================
  Files          184      184           
  Lines        24141    24141           
  Branches      3123     3123           
========================================
  Hits         22263    22263           
  Misses        1813     1813           
  Partials        65       65           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c8abbee...fd3cbf5. Read the comment docs.

@orbeckst orbeckst merged commit 56cb47c into develop Jul 8, 2020
@orbeckst orbeckst deleted the add-privacy-policy branch July 8, 2020 21:37
@orbeckst
Copy link
Member Author

orbeckst commented Jul 9, 2020

@lilyminium where do the development docs live at the moment? The usual https://www.mdanalysis.org/mdanalysis redirects to the https://docs.mdanalysis.org/1.0.0/ docs and the 1.0.0 docs do not have a version selector. I’d like to see if the privacy footer properly displays.

I’ll still have to backport to 1.0.x #2798 but I assume we’ll have to push these docs manually?

@lilyminium
Copy link
Member

lilyminium commented Jul 9, 2020 via email

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.

add GoatCounter tracking to docs

5 participants