Skip to content

doc(analytics): add README.md for GA4 analytics#21142

Open
criticalAY wants to merge 1 commit into
ankidroid:mainfrom
criticalAY:docs/analytics
Open

doc(analytics): add README.md for GA4 analytics#21142
criticalAY wants to merge 1 commit into
ankidroid:mainfrom
criticalAY:docs/analytics

Conversation

@criticalAY
Copy link
Copy Markdown
Contributor

Purpose / Description

Add documentation for analytics and the library related to it

Fixes

Approach

NA

How Has This Been Tested?

NA

Learning (optional, can help others)

NA

Checklist

Please, go through these checks before submitting the PR.

  • You have a descriptive commit message with a short title (first line, max 50 chars).
  • You have commented your code, particularly in hard-to-understand areas
  • You have performed a self-review of your own code
  • UI changes: include screenshots of all affected screens (in particular showing any new or changed strings)
  • UI Changes: You have tested your change using the Google Accessibility Scanner

Copy link
Copy Markdown
Member

@david-allison david-allison left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

Comment thread docs/analytics/README.md
# Analytics

AnkiDroid sends opt-in anonymous usage data to Google Analytics 4 (GA4). The
transport is [google-analytics-kt](https://github.com/criticalAY/google-analytics-kt) see its [README](https://github.com/criticalAY/google-analytics-kt) for the
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

These two links are the same

Comment thread docs/analytics/README.md
Comment on lines +110 to +112
For getting a measurement ID + API secret from the GA4 admin panel, follow
the library's [Prerequisites section][lib-prereqs]. Once you have them, plug
them into AnkiDroid like so:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

[potentially for a follow-up, don't want to block], this doesn't produce enough detail for someone to follow the steps

@david-allison david-allison added Needs Second Approval Has one approval, one more approval to merge and removed Needs Review labels May 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs Second Approval Has one approval, one more approval to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants