Skip to content

Conversation

gabrielcaires
Copy link
Contributor

@gabrielcaires gabrielcaires commented Aug 21, 2025

Part of CML-805

Proposed Changes

  • Track the algo during like actions

Why are these changes being made?

  • We want to have visibility into the number of posts liked, which comes from the

Testing Instructions

Scenario: Like in the freshly-pressed tab

  • Go to /reader?flags=reader/discover/freshly-pressed
  • Go to /discover >Frehsly pressed tab
  • Click on the like button and check if the calypso_reader_article_liked with the algo value = 'freshly-pressed'

Scenario: Like in a post from a freshly-pressed tab

  • Go to /reader?flags=reader/discover/freshly-pressed
  • Go to /discover >Frehsly pressed tab
  • Click in any post
  • Click on the like button
  • Click on the like button and check if the calypso_reader_article_liked with the algo value = 'freshly-pressed'

Pre-merge Checklist

  • Has the general commit checklist been followed? (PCYsg-hS-p2)
  • Have you written new tests for your changes?
  • Have you tested the feature in Simple (P9HQHe-k8-p2), Atomic (P9HQHe-jW-p2), and self-hosted Jetpack sites (PCYsg-g6b-p2)?
  • Have you checked for TypeScript, React or other console errors?
  • Have you tested accessibility for your changes? Ensure the feature remains usable with various user agents (e.g., browsers), interfaces (e.g., keyboard navigation), and assistive technologies (e.g., screen readers) (PCYsg-S3g-p2).
  • Have you used memoizing on expensive computations? More info in Memoizing with create-selector and Using memoizing selectors and Our Approach to Data
  • Have we added the "[Status] String Freeze" label as soon as any new strings were ready for translation (p4TIVU-5Jq-p2)?
    • For UI changes, have we tested the change in various languages (for example, ES, PT, FR, or DE)? The length of text and words vary significantly between languages.
  • For changes affecting Jetpack: Have we added the "[Status] Needs Privacy Updates" label if this pull request changes what data or activity we track or use (p4TIVU-aUh-p2)?

@gabrielcaires gabrielcaires changed the title Record algo during the like action Reader; Record algo during the like action Aug 25, 2025
@gabrielcaires gabrielcaires changed the title Reader; Record algo during the like action Reader: Record algo during the like action Aug 25, 2025
@gabrielcaires gabrielcaires marked this pull request as ready for review August 25, 2025 09:45
@gabrielcaires gabrielcaires requested a review from a team as a code owner August 25, 2025 09:45
@matticbot matticbot added the [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. label Aug 25, 2025
@gabrielcaires gabrielcaires self-assigned this Aug 25, 2025
@gabrielcaires gabrielcaires merged commit 869f33c into trunk Aug 26, 2025
14 of 16 checks passed
@gabrielcaires gabrielcaires deleted the update/improve-tracking branch August 26, 2025 07:41
@github-actions github-actions bot removed the [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. label Aug 26, 2025
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.

3 participants