search

Developer-DAO / academy-turbo

D_D Academy is an open-source education platform created by and for Developer DAO.
https://academy.developerdao.com
MIT License
18 stars 10 forks source link

feat: adding vercel analytics #192

Closed Markkos89 closed 8 months ago

Markkos89 commented 8 months ago

Changes

vercel[bot] commented 8 months ago

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

Name Status Preview Comments Updated (UTC)
academy-turbo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 23, 2024 2:01pm
academy-turbo-storybook ✅ Ready (Inspect) Visit Preview Feb 23, 2024 2:01pm
github-actions[bot] commented 8 months ago

Running Lighthouse audit...

kempsterrrr commented 8 months ago

As commented here in discord, before okaying such things please ensure the potential cost is discussed and considered.

In this case, our sponsored Vercel Pro accounts gives us up to 25k events for free , after that it is 14 USD per 100k events:

image

Analytics was turned on for the old version of academy, in January the project recorded 1377 analytics events with 0 promotion for courses:

image

As we start to promote courses this number could expand rapidly so whilst I think it's fine for us to deploy this now we need to keep an eye on these numbers as it can get more expensive pretty quick and Academy at the min is pre-revenue. This highlights the importance of prioritising binging in money for this project.

For anything that requires a cost I'm happy to be the filter for OK'ing. Please make sure these points are considered before signing things off that might cost money in the future.

TLDR happy to see this merged, please can we decide/confirm a) who is keeping an eye on these usage stats + therefore costs, b) agree the sign-off for things that come with a cost c)

@elPiablo @Markkos89 @wolovim

Markkos89 commented 8 months ago

Hey @kempsterrrr thank you for bringin this to the table and for the insights. I up to change the analytics implementation to any other service of preference, mate! Really up to it. I just need the analytics, looking to Sentry and Hotjar to.

There is no rush to merge this PR if other option for analytics appears

kempsterrrr commented 8 months ago

I think it's fine to merge this at this moment in time as it's free @Markkos89 . The main point I am making is the need to consider the cost before we introduce tooling or deploy things :)

Sentry and Hotjar as far as I understand solve different problems - error reporting and on page use tracking/analytics.

My suggestion would be merge this and then consider these alternative tools and open a separate issue for them with costs etc. considered

kempsterrrr commented 8 months ago

@Markkos89 feel free to merger this PR. Its important we have some analytics in place we just need to keep an eye on usage is all :)