Closed maximilianoertel closed 1 month ago
Status | Category | Percentage | Covered / Total |
---|---|---|---|
🔵 | Lines | 1.22% | 92 / 7536 |
🔵 | Statements | 1.18% | 98 / 8291 |
🔵 | Functions | 1.09% | 20 / 1830 |
🔵 | Branches | 0.68% | 31 / 4509 |
Visit the preview URL for this PR (updated for commit 04f7c69):
https://roar-staging--pr782-ref-318-query-compos-y63fmfz2.web.app
(expires Fri, 13 Sep 2024 21:57:51 GMT)
🔥 via Firebase Hosting GitHub Action 🌎
Sign: 2631e9c58fd0104ecbfddd72a62245ddac467460
Passed #5958
•
04f7c6989e: E2E Tests for PR 782 "Migrate Administrator homepage to composable TanStack quer...
Project |
roar-dashboard-e2e
|
Branch Review |
ref/318/query-composables-home-administrator
|
Run status |
Passed #5958
|
Run duration | 04m 19s |
Commit |
04f7c6989e: E2E Tests for PR 782 "Migrate Administrator homepage to composable TanStack quer...
|
Committer | Maximilian Oertel |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
0
|
Flaky |
0
|
Pending |
0
|
Skipped |
0
|
Passing |
32
|
View all changes introduced in this branch ↗︎ |
Proposed changes
This PR updates the
Home Administrator
page to use composable TanStack queries.Following the recently merged PR #749, this PR also includes some optimisations in regards to loading and fetching of administrations data. Leveraging the newly introduced
useAdministrationsQuery
composable, we can toggle between regular and test data and display the loading state when switching. Once fetched, data is kept in cache/memory until garbage collection time.As the added reactivity inside the query composable adds additional complexity, this PR does not yet include unit tests. Those will be added as part of a separate PR.
https://github.com/user-attachments/assets/2cecd306-4bfa-426d-9883-71d8f91c64ac
Types of changes
What types of changes does this pull request introduce?
Checklist
Justification of missing checklist items
Further comments
Ref https://github.com/yeatmanlab/roar/issues/318