-
Notifications
You must be signed in to change notification settings - Fork 43
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor Projects Dashboard & Fix Last Activity Bug #2606
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2606 +/- ##
==========================================
+ Coverage 90.79% 90.84% +0.04%
==========================================
Files 318 320 +2
Lines 11178 11215 +37
==========================================
+ Hits 10149 10188 +39
+ Misses 1029 1027 -2 ☔ View full report in Codecov by Sentry. |
2ec29c8
to
679aa9f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great component building! 💯
Description
This PR refactors the projects dashboard page by encapsulating the three different components of the page into separate live components. This change makes the page more reusable, improves separation of concerns, and ensures the page itself doesn't handle state and events for these components. Additionally, this PR also fixes a bug identified in the last activity column of the projects list. The bug was due to the last activity column displaying the value of the updated_at column from the projects table. The query has been updated to fetch the most recent updated_at value of the workflows for each project.
Fixes #2593
Validation steps
Additional notes for the reviewer
AI Usage
Please disclose how you've used AI in this work (it's cool, we just want to know!):
You can read more details in our Responsible AI Policy
Pre-submission checklist
:owner
,:admin
,:editor
,:viewer
)