Skip to content

SER-2859: Use consistent 'Push Actioned' label across Drill UI#7597

Open
mrmeghana wants to merge 2 commits into
masterfrom
mrmeghana-push-actioned-label-consistency
Open

SER-2859: Use consistent 'Push Actioned' label across Drill UI#7597
mrmeghana wants to merge 2 commits into
masterfrom
mrmeghana-push-actioned-label-consistency

Conversation

@mrmeghana
Copy link
Copy Markdown
Contributor

@mrmeghana mrmeghana commented May 26, 2026

Summary

SER-2859

In the Drill UI, the push event was labelled inconsistently:

  • The event button under QUERY BASED ON said "Push action" (from i18n internal-events.[CLY]_push_action)
  • The Push tab in the SEGMENTATION row said "Push Actioned" (hardcoded in helpers.js)

Change

  • helpers.js: use the i18n key as the source of truth for the tab label (matching how Journey and other tabs already work). Replaces a hardcoded "Push Actioned".
  • push.properties (English default): change the canonical value from Push action to Push Actioned so both UI spots render the preferred wording in English.

Non-English locale files are left untouched — translators have already chosen the correct local term for internal-events.[CLY]_push_action, and since both UI spots now read from the same key, they will automatically stay in sync per-locale.

Test plan

  • Open Drill in the dashboard
  • Pick Push Actioned from the segmentation tab row
  • Confirm the event button under "QUERY BASED ON" also shows Push Actioned (no more "Push action" mismatch)
  • Switch to a non-English locale and confirm both spots show the same (locale-translated) wording
  • Regression: Journey, Crash, Sessions tabs still work normally

mrmeghana and others added 2 commits May 26, 2026 16:24
In the Drill segmentation row, the Push tab was hardcoded as
"Push Actioned" while the event button under "QUERY BASED ON"
showed "Push action" (from the i18n string). Two different
wordings for the same entity.

Use the i18n key as the source of truth for the tab label
(matching how Journey and other tabs work) and update the
English default to "Push Actioned" so both spots stay in sync
via a single string.

Other-language locale files keep their existing translations.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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.

2 participants