Skip to content

[IP-694]: Fix recent dashboard widget ordering - #699

Draft
PuvaanRaaj wants to merge 1 commit into
InvoicePlane:developfrom
PuvaanRaaj:fix/694-recent-widget-ordering
Draft

[IP-694]: Fix recent dashboard widget ordering#699
PuvaanRaaj wants to merge 1 commit into
InvoicePlane:developfrom
PuvaanRaaj:fix/694-recent-widget-ordering

Conversation

@PuvaanRaaj

@PuvaanRaaj PuvaanRaaj commented Aug 1, 2026

Copy link
Copy Markdown

Summary

  • order Recent Expenses, Payments, Projects, and Tasks by descending primary key
  • avoid Laravel’s default created_at ordering on models that disable timestamps
  • add ordered-record regression coverage for each widget

Developer checklist

  • References CHECKLIST.md rows: expenses, payments, projects, and projects/tasks
  • Includes appropriate test coverage
  • Follows service/DTO/transformer structure (no new inline logic)
  • UI follows Filament & Livewire best practices
  • Translations not needed
  • Full php artisan test suite not run
  • Ran vendor/bin/pint for changed files

Validation

  • php artisan test Modules/Expenses/Tests/Feature/RecentExpensesWidgetTest.php Modules/Payments/Tests/Feature/RecentPaymentsWidgetTest.php Modules/Projects/Tests/Feature/RecentProjectsWidgetTest.php Modules/Projects/Tests/Feature/RecentTasksWidgetTest.php (8 tests, 16 assertions; existing PHP 8.5 deprecations only)
  • vendor/bin/pint --test on the eight changed files

Closes #694

Summary by CodeRabbit

  • Bug Fixes

    • Recent expenses, payments, projects, and tasks now consistently appear newest first.
    • Widgets display the latest records in the expected order.
  • Tests

    • Added coverage confirming correct ordering and successful rendering for all recent activity widgets.

Use the primary key for timestamp-less dashboard models and cover descending record order across the affected widgets.

Fixes InvoicePlane#694
@InvoicePlane InvoicePlane deleted a comment from coderabbitai Bot Aug 1, 2026
@nielsdrost7

nielsdrost7 commented Aug 1, 2026

Copy link
Copy Markdown
Collaborator

Hey @PuvaanRaaj thanks for the PR, man!
You opened the PR towards InvoicePlane:develop
But the correct repository is InvoicePlane-v2:develop

@nielsdrost7
nielsdrost7 marked this pull request as draft August 1, 2026 06:48
@nielsdrost7 nielsdrost7 changed the title [PR] Fix recent dashboard widget ordering [IP-694]: Fix recent dashboard widget ordering Aug 1, 2026
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.

[Tests]: Recent*Widget ordering fix has no regression test

2 participants