Closed maximilianoertel closed 1 day ago
Status | Category | Percentage | Covered / Total |
---|---|---|---|
🔵 | Lines | 1% | 73 / 7291 |
🔵 | Statements | 0.92% | 74 / 8013 |
🔵 | Functions | 0.79% | 14 / 1757 |
🔵 | Branches | 0.45% | 20 / 4365 |
Visit the preview URL for this PR (updated for commit 368a986):
https://roar-staging--pr748-ref-318-query-compos-licip3pp.web.app
(expires Tue, 22 Oct 2024 22:12:49 GMT)
🔥 via Firebase Hosting GitHub Action 🌎
Sign: 2631e9c58fd0104ecbfddd72a62245ddac467460
Passed #8431
•
368a98652e: E2E Tests for PR 748 "`CU-8689c0g67` Migrate TanStack queries to query composabl...
Project |
roar-dashboard-e2e
|
Run status |
Passed #8431
|
Run duration | 03m 28s |
Commit |
368a98652e: E2E Tests for PR 748 "`CU-8689c0g67` Migrate TanStack queries to query composabl...
|
Committer | Adam Richie-Halford |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
0
|
Flaky |
0
|
Pending |
0
|
Skipped |
0
|
Passing |
14
|
Proposed changes
This PR migrates the existing TanStack queries defined throughout the codebase to re-usable query composables.
Key Highlights:
Additional Notes
package-lock
file due to build failures in the CI caused by npm/cli#4828.@tanstack/vue-query
to the latest version, which introduces some deprecations.cacheTime
) and stale time.getAxiosInstance
helper with abaseUrl
check to ensure it is always set, addressing an issue that appears to stem from the initialization process.useDistrictsQuery
composable to enable data caching and reuse.<HomeSelector />
component now lazy loads the homepage and consent modal components.Chained Pull Requests:
751
757
760
769
775
776
777
782
785
791
794
804
815
816
832
835
837
840
845
867
868
870
882
Types of changes
Checklist
Justification of missing checklist items
n/a
Further comments
n/a
Resolves https://github.com/yeatmanlab/roar/issues/318 Resolves https://github.com/yeatmanlab/roar/issues/376