openedx / wg-data

Tracking work and progress of the Open edX Data Working Group
1 stars 2 forks source link

Docs: Move developer guide analytics page #23

Closed bmtcril closed 21 hours ago

bmtcril commented 2 years ago

This page has a combination of very relevant useful information and outdated / incorrect information. We'd like to move it into docs.openedx.org! This will include reformatting it into several different docs, which may include duplicating some content. This is ok! Here are some docs on how to get started writing docs: https://docs.openedx.org/en/latest/documentors/index.html

Original doc: https://edx.readthedocs.io/projects/edx-developer-guide/en/latest/analytics.html

Bonus doc, which has some edx.org oriented additional flavor text that may or may not be useful to bring in as this gets fleshed out: https://openedx.atlassian.net/wiki/spaces/DE/pages/2919432202/Analytical+event+flow+2021

Things to add:

Structural updates:

Content that will need updating off the top of my head:

6.1.1. Javascript event emission, I'm not sure how true this is now or what the current state of this emission is, we should ask the frontend working group and update as necessary.

6.1.1.5. and any other devstack references

6.1.1.6.2. The test code here is very old and should be updated to be more modern (no classes, pytest, etc)

6.1.1.6.3. Bokchoy references can be removed

6.1.2. Will need to be updated to point at the new docs

6.1.4. I guess this situation is not temporary.

e0d commented 1 year ago

@bmtcril is this still reasonably accurate?

https://openedx.atlassian.net/wiki/spaces/AN/pages/13959562/Code+Snippets

bmtcril commented 1 year ago

@e0d yes-ish. I think it needs a refresh for Tutor and how MFEs handle sending events, which I think is fairly different than doing it through the LMS frontend code. I think it needs some more context around the overall event lifecycle as above, too, but could definitely be used as a base for the updated doc.