feat(affiliate): show the exact updated at value for stats#7169
feat(affiliate): show the exact updated at value for stats#7169kernelwhisperer merged 1 commit intomainfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
WalkthroughThe pull request extends the MetricsCard component in the affiliate module to accept and display a Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 1 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment Tip CodeRabbit can generate a title for your PR based on the changes.Add |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@apps/cowswap-frontend/src/modules/affiliate/pure/MetricsCard.tsx`:
- Around line 89-91: The relative "last updated" text (statsUpdatedTimeAgo) is
still hiding the exact timestamp in the title attribute
(formatDateWithTimezone(statsUpdatedAt)), which is inaccessible on
touch/keyboard; change the span that currently uses
title={formatDateWithTimezone(statsUpdatedAt)} to render both the relative time
and a visible exact timestamp (for example show "{statsUpdatedTimeAgo} —
{formatDateWithTimezone(statsUpdatedAt)}" or place the exact value in a visible
<time> element with dateTime={statsUpdatedAt}) so the exact updated-at is always
readable; keep the existing HelpTooltip usage and ensure the date/time element
uses the same formatDateWithTimezone(statsUpdatedAt) value for consistency and
accessibility.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
Run ID: d6cd6692-74b5-4484-8286-9363a74f6a62
📒 Files selected for processing (3)
apps/cowswap-frontend/src/modules/affiliate/containers/AffiliatePartnerStats.tsxapps/cowswap-frontend/src/modules/affiliate/containers/AffiliateTraderStats.tsxapps/cowswap-frontend/src/modules/affiliate/pure/MetricsCard.tsx





Summary
Now that we moved away from 6hrs to 24hrs we can show the exact value of updated at.
Summary by CodeRabbit
New Features
Improvements