Skip to content

update advanced analytics#115

Merged
iamshadmirza merged 3 commits into
mainfrom
shad/remove-umami-analytics
Apr 4, 2024
Merged

update advanced analytics#115
iamshadmirza merged 3 commits into
mainfrom
shad/remove-umami-analytics

Conversation

@iamshadmirza

Copy link
Copy Markdown
Contributor

We need to stop sending data to umami as it's deprecated.
Since we upgraded our analytics usage to Kinesis, we will send data to our own infra.

This PR takes care of removing older usage and send correct data to newer analytics system.

@vercel

vercel Bot commented Apr 4, 2024

Copy link
Copy Markdown

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

Name Status Preview Comments Updated (UTC)
starter-kit ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 4, 2024 2:40pm

@iamshadmirza

Copy link
Copy Markdown
Contributor Author

Note: We don't have series page in personal theme so I didn't include seriesId while sending analytics data (only in personal theme).

@JannikWempe JannikWempe left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't see how this should have been working earlier due to the error pointed out.

The comments to hashnode theme apply to the other themes as well.

Comment thread packages/blog-starter-kit/themes/hashnode/components/analytics.tsx Outdated
Comment thread packages/blog-starter-kit/themes/hashnode/components/analytics.tsx Outdated
Comment thread packages/blog-starter-kit/themes/hashnode/components/analytics.tsx
@iamshadmirza iamshadmirza changed the title update umami analytics update advanced analytics Apr 4, 2024
@iamshadmirza iamshadmirza merged commit f543e14 into main Apr 4, 2024
@iamshadmirza iamshadmirza deleted the shad/remove-umami-analytics branch April 4, 2024 17:48
hoangsvit pushed a commit to hoangsvit/hashnode-starter-kit that referenced this pull request Jul 15, 2025
* update umami analytics

* update gql schema

* refactor and misc fixes
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