Repository navigation
fix: describe mention activities as mentions of the project - #4999
Conversation
Mention activities read "mentioned you on Reddit" or "mentioned you in a tweet", but in CDP they are mentions of an LF project, whether they come from Octolens or from a project's connected X/Twitter account. The display and short labels now say "mentioned the project". Refs linuxfoundation/lfx-cdp-insights-ops#748 Signed-off-by: Uroš Marolt <uros@marolt.me>
PR SummaryLow Risk Overview The shared Display text only—activity types, scoring, and contribution behavior are unchanged. Reviewed by Cursor Bugbot for commit e29da56. Bugbot is set up for automated code reviews on this repo. Configure here. |
There was a problem hiding this comment.
🟢 Approval recommended
The changes consistently update display strings without changing behavior, and no blocking issues were identified.
0 open findings
What changed in this PR
Updates CDP mention labels to describe mentions of the project rather than the user.
Changes:
- Updates platform-specific and shared mention display text and filter labels.
- Aligns the X/Twitter pretty label without changing activity identifiers, scores, or contribution flags.
| File | Description |
|---|---|
| services/libs/integrations/src/integrations/prettyActivityTypes.ts | Updates the X/Twitter mention label. |
| services/libs/integrations/src/integrations/activityTypes.ts | Updates shared and platform-specific mention labels. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Summary
Mention activities were described as "mentioned you on Reddit", "mentioned you on GitHub" or "mentioned you in a tweet". In CDP these are mentions of an LF project, both the ones Octolens now ingests and the ones from a project's connected X/Twitter account, so the display text and the short label (used by the activity-type filter) now say "mentioned the project".
Changes
mentiondisplay text and short label for every platform that has one: GitHub, X/Twitter, LinkedIn, Reddit, DEV, Hacker News, Stack Overflow, Medium, Product Hunt, Facebook, YouTube, Bluesky, TikTok and Instagram.prettyActivityTypesmatches.activityTypestable's existing(mention, twitter)label ("Mentioned a user") is left as is.Type of change
Related issue
Refs linuxfoundation/lfx-cdp-insights-ops#748