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.

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

Add editorial metadata analytics #64

Merged
merged 4 commits into from
Oct 14, 2024

Conversation

alecgeatches
Copy link
Contributor

Description

This adds some VIP-only telemetry for editorial metadata fields:

  • em_field_created, with data:
    • int term_id
    • string name
    • string slug
    • string type (e.g. text/checkbox)
  • em_field_changed, with data (same as above):
    • int term_id
    • string name
    • string slug
    • string type (e.g. text/checkbox)
  • em_field_deleted, with data (same as above):
    • int term_id
    • string name
    • string slug
    • string type (e.g. text/checkbox)

Additionally in #21 we also record these data related to EM fields:

  • In record_add_custom_status(), which fires when a new custom status field is created, we record the number of users and number of required EM fields associated with that custom status.

  • In record_update_custom_status(), which fires each time a custom status is edited in the Workflow editor, we record the number of users and number of required EM fields associated with that custom status.

Between these two sets of data, we should be able to see if folks are doing CRUD operations on EM fields as well as how many are being used in the required functionality of custom statuses.

@alecgeatches alecgeatches self-assigned this Oct 14, 2024
@alecgeatches alecgeatches requested a review from a team as a code owner October 14, 2024 18:29
@alecgeatches alecgeatches merged commit 53259df into trunk Oct 14, 2024
5 checks passed
@alecgeatches alecgeatches deleted the add/editorial-metadata-analytics branch October 14, 2024 22:16
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.

1 participant