fixed issue with the age indicator#11
Conversation
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughSmall, focused update to issue age and last-seen metric display in the issues dialog. Age now computes from first_seen only with updated subtitle text, and last-seen removes inline styling while maintaining time-ago display. ChangesIssue metrics display
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Warning Review ran into problems🔥 ProblemsGit: Failed to clone repository. Please run the 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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@src/Views/dex/issues_dialog_shell.php`:
- Line 123: In the Age subtitle (the div with class "dex-stat__sub") fix the
spelling and trailing space: change the text "first occurence → now " to "first
occurrence → now" by correcting "occurence" to "occurrence" and removing the
extra space after "now"; locate the element using the dex-stat__sub class to
update the string.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 4de46857-e06e-4f76-947f-53ecc92d2179
📒 Files selected for processing (1)
src/Views/dex/issues_dialog_shell.php
|
@coderabbitai resolve |
✅ Actions performedComments resolved and changes approved. |
Summary
The age indicator on the Issues view is wrong. It calculates from first occurence to last occurence. Should be first occurence to now.
How to test
View an issue detail