yeatmanlab / roar-dashboard

A dashboard to administer ROAR assessments
https://roar.education
Other
4 stars 4 forks source link

Migrate List Organisations page to composable TanStack queries #760

Closed maximilianoertel closed 1 month ago

maximilianoertel commented 1 month ago

Proposed changes

This PR migrates the List Organisations to use new composable TanStack queries. Whilst there remains room for improvement on the query side, this should be addressed in a separate ticket/PR.

Types of changes

Checklist

Justification of missing checklist items

n/a

Further comments

n/a

Ref https://github.com/yeatmanlab/roar/issues/318

github-actions[bot] commented 1 month ago

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 1.22% 92 / 7539
🔵 Statements 1.18% 98 / 8294
🔵 Functions 1.09% 20 / 1829
🔵 Branches 0.68% 31 / 4512
File Coverage
File Stmts % Branch % Funcs % Lines Uncovered Lines
Changed Files
src/components/ListOrgs.vue 0% 0% 0% 0% 136-142, 144-149, 145-147, 146, 148, 151-156, 152-154, 153, 155, 158-159, 161-163, 165-166, 166, 168-195, 169-174, 176, 176, 178-183, 180-182, 184-187, 185-186, 188-190, 189, 191-193, 192, 194, 197-200, 199, 202, 202, 204-206, 208-210, 209, 212-214, 220-222, 225-242, 228-233, 236-241, 245-255, 246-254, 273-336, 274-335, 276, 278-286, 279-285, 288-296, 289-295, 299, 301-316, 301-316, 319, 321-326, 328-334, 338-386, 339-344, 346-351, 347-350, 353-356, 354-355, 358-383, 385, 388-401, 389, 389-400, 391-399, 403-409, 404-408, 406-407, 411-413, 412, 416-419, 417, 417-418, 421-423, 422, 422, 425-427, 426, 426, 429-431, 430, 433-435, 434, 437-440, 439, 3-4, 1, 16, 21, 33, 37, 49, 70, 78, 80, 92, 95, 102, 105, 1-14, 1-23, 1-39, 63, 1-72, 90, 100
src/composables/queries/useOrgsTableQuery.js 0% 0% 0% 0% 22-43, 23-24, 26, 26, 28-42, 31-40
src/constants/auth.js 100% 100% 100% 100%
src/constants/queryKeys.js 0% 100% 100% 0% 1-10
src/helpers/query/orgs.js 0% 0% 0% 0% 17-153, 40-44, 46-55, 47-50, 48-49, 52-54, 53, 57-62, 64-136, 65-136, 73-103, 74-94, 96-102, 104-136, 105-135, 106-126, 128-134, 138-150, 139-149, 152, 155-229, 156-228, 157-170, 169, 172-175, 174, 177, 179-227, 181-183, 182, 185-187, 186, 189-190, 192, 192-193, 193-227, 196-215, 197-212, 198-212, 200-212, 203-205, 204, 207-211, 210, 214, 216-227, 217-225, 218-224, 219-222, 221, 223, 226, 231-244, 232-241, 243, 243, 246-332, 253-331, 254-261, 263-267, 264-267, 266, 269, 269, 271-330, 272-274, 273, 275-330, 278-280, 279, 283-285, 284, 287-289, 288, 291-293, 293-294, 294, 296-298, 297, 300-330, 302-327, 304-306, 305, 307-327, 309-312, 311, 313-327, 315-317, 316, 318-322, 321, 323-325, 324, 326, 329, 334-383, 344-354, 356, 358-382, 359, 359, 361-369, 362, 362-369, 368, 368, 371, 373, 373-380, 377, 377-378, 378-379, 381, 385-420, 394-403, 405-419, 406, 406, 408-418, 429-555, 430-433, 432, 432, 435-440, 438, 438, 442, 444, 446-469, 447-467, 459-467, 460-466, 468, 471, 471, 473-474, 474, 476, 478-480, 479, 482-484, 483, 486, 486-489, 488, 491-494, 496, 498-515, 500, 502, 502, 504-512, 517, 517-518, 520-537, 521-522, 522-536, 524-533, 525-530, 532, 535, 539-540, 540-541, 541, 543-552, 545-551, 546-550, 547, 547-548, 548-549, 554
Generated in workflow #244
github-actions[bot] commented 1 month ago

Visit the preview URL for this PR (updated for commit 26aff29):

https://roar-staging--pr760-ref-318-query-compos-w7zwta21.web.app

(expires Mon, 16 Sep 2024 18:46:52 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 2631e9c58fd0104ecbfddd72a62245ddac467460

cypress[bot] commented 1 month ago

roar-dashboard-e2e    Run #5997

Run Properties:  status check passed Passed #5997  •  git commit 26aff29c80: E2E Tests for PR 760 "Migrate List Organisations page to composable TanStack que...
Project roar-dashboard-e2e
Branch Review ref/318/query-composables-list-orgs
Run status status check passed Passed #5997
Run duration 05m 33s
Commit git commit 26aff29c80: E2E Tests for PR 760 "Migrate List Organisations page to composable TanStack que...
Committer Maximilian Oertel
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 32
View all changes introduced in this branch ↗︎