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.

Sign up for GitHub

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

Jump to bottom

Feature/error analytics microservice #34

Merged
merged 53 commits into from
Dec 15, 2023
Merged

Feature/error analytics microservice #34

merged 53 commits into from
Dec 15, 2023

Conversation

buckhalt
Copy link
Member

@buckhalt buckhalt commented Nov 17, 2023 •
edited
Loading

This PR implements error and event reporting to our custom microservice using the package @codaco/analytics https://github.com/complexdatacollective/error-analytics-microservice

@buckhalt
demonstration of plausible with goals and custom props
c97b8d8
@buckhalt
add installationid field and use for plausible
7abb56a
@buckhalt
disable automatic pageview tracking
5cafdcd
@buckhalt
implement ability for researchers to disaBle analytics
1101ea3
@buckhalt
wip add analytics switch to onboarding flow
507b082
@buckhalt
working set analytics from onboard
7089886 
uses refetch to ensure ui reflects current data. this is causing an issue with the steps routing as it reroutes to step 2 when it revalidates
@buckhalt
fix issue where onboard was redirecting to step 2
aa4bf5e
@buckhalt
enabled is false if setting is disabled
fc797be 
also adds commented out props for when we switch to selfhosted
@buckhalt
custom useAnalytics hook, move plausibleprovider to proviers
255e821
@buckhalt
switch to self hosted
9f718b7
@buckhalt
wrap trackEvent in check for appSettings, geo ip stuff
2c2ecc1 
uses maxmind to get isocode from ip address. wip get ip from headers
@buckhalt
wip send errors using analytics pacakge
3e52369
@buckhalt
move reporting logic within Error component
86f8afb
@buckhalt
sendError func to wrap trackError
1056e0f
@buckhalt
install published analytics package
d765098
@buckhalt
working send event, wip send error
062a68a 
need to be in server components so trying to use trpc for errors
@buckhalt
working error reporting
c6d5f68 
sending twice though. maybe due to react strict mode?
@buckhalt
rm plausible
75c7366
@buckhalt
rm plausible provider
b6a9dcb
@vercel Vercel
Copy link

vercel bot commented Nov 17, 2023 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
fresco ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 15, 2023 6:36pm

@buckhalt
maxmind integration
ce7e771 
use the class constructor with global var pattern to create single global instance of analytics client class, use route handler helper to get geolocation
@buckhalt
wip enabled as default
cd68bfa 
events queue not running for some reason
@buckhalt
manually merge 'main' package.json
c512826 
also fixes linting error
@buckhalt buckhalt marked this pull request as ready for review December 15, 2023 18:37
@buckhalt buckhalt merged commit 96e87e0 into main Dec 15, 2023
3 checks passed
@jthrilly jthrilly deleted the feature/error-analytics-microservice branch April 5, 2024 10:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@buckhalt @jthrilly