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

Add analytics #346

Merged
merged 7 commits into from
Nov 7, 2023
Merged

Add analytics #346

merged 7 commits into from
Nov 7, 2023

Conversation

thostetler
Copy link
Member

  • add react-gtm-hook
  • add analytics call for theme change
  • Add analytics call for search example click
  • add analytics call for facets

@thostetler thostetler marked this pull request as ready for review November 3, 2023 21:01
@thostetler thostetler requested a review from shinyichen November 3, 2023 21:01
Copy link

codecov bot commented Nov 3, 2023

Codecov Report

Merging #346 (fce662b) into master (52142e0) will decrease coverage by 0.06%.
The diff coverage is 11.86%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #346      +/-   ##
==========================================
- Coverage   47.89%   47.83%   -0.06%     
==========================================
  Files         392      392              
  Lines       40555    40611      +56     
  Branches      558      558              
==========================================
+ Hits        19422    19428       +6     
- Misses      21121    21171      +50     
  Partials       12       12              
Files Coverage Δ
src/components/SearchFacet/SearchFacet.tsx 16.90% <14.28%> (-0.05%) ⬇️
src/components/SearchExamples/SearchExamples.tsx 21.42% <12.50%> (-0.80%) ⬇️
src/components/AllAuthorsModal/AllAuthorsModal.tsx 19.62% <7.69%> (-0.44%) ⬇️
...ponents/CitationExporter/components/ResultArea.tsx 55.84% <18.75%> (-9.73%) ⬇️
src/components/CopyButton/CopyButton.tsx 22.03% <6.66%> (-4.06%) ⬇️

... and 1 file with indirect coverage changes

@thostetler thostetler merged commit 5093a39 into adsabs:master Nov 7, 2023
3 of 4 checks passed
@thostetler thostetler deleted the analytics branch November 8, 2023 14:13
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