Bug/WP-798: Setup Defaults for Query Client #395

jalowe13 · 2024-12-06T20:37:02Z

Overview

Setup defaults for query client
…

Changes

Defaults for QueryClientProvider wrapper around Components

Testing

Add the following line to src/client/src/main.tsx:32

console.log(queryClient.getQueryCache());

Navigate to a new page and check in the console if the Query options match the following
{staleTime: 300000, cacheTime: 600000, refetchOnWindowFocus: false, queryKey: Array(1), queryFn: ƒ, ...}

UI

…

…276) * Initial commit - using react with APCD page * remove visualizer * Remove print statements * Renamed shared-components to core-components

…-Custom into feat/apcd-react-conversion

* apcd admin submissions react page version 1 * moved filter and sort functionality to react * Added View logs modal using reactstrap modal --------- Co-authored-by: Chandra Y <cyemparala@tacc.utexas.edu>

* feat: Add ViewUsers component * chore: Update Docker Compose dev configuration for React * refactor: Update ViewUsers component and related files * refactor: Update ViewUsers component and related files * feat: Add user action dropdown in ViewUsers component * Filter by Status and Org now work * URL updated w/ status and org * URL updates to show status and/or organization --------- Co-authored-by: Chandra Y <cyemparala@tacc.utexas.edu>

…react (#294) * Setup vite local and template with react content * Update Makefile

Modal windows not included

) * List registrations: filters, pagination and view modal * Add pending edit modal and fix org status

* first run at the react conversion * Update list_admin_extension.html from merge feat branch * Updated incorrect import --------- Co-authored-by: Chandra Y <cyemparala@tacc.utexas.edu>

* feat: Add user action dropdown in ViewUsers component * Add ViewRecordModal component and styles for user details modal * Update ViewRecordModal component and styles for user details modal * Update ViewRecordModal component and styles for user details modal * Create Edit Record modal * Add Success and Error modals * Update EditRecordModal component and fix error handling * Add support for fetchUtil request method and use it in EditRecordModal * Add update user method * Add pagination to ViewUsersTable component * Update ViewUsersTable component with duplicate path in urls.py * Update EditRecordModal component and fix error handling * Remove link --------- Co-authored-by: Chandra Y <cyemparala@tacc.utexas.edu>

…Core-CMS-Custom into feat/apcd-react-conversion

* Update RegistrationList and ViewRegistrationModal to display 'None' when year, reg_status, fein, license, or naic is undefined or null --------- Co-authored-by: Chandra Y <cyemparala@tacc.utexas.edu>

…Core-CMS-Custom into feat/apcd-react-conversion

Co-authored-by: Chandra Y <cyemparala@tacc.utexas.edu>

* WP-795: clear options button similarity across all pages * set styles

* Default filter admin submissions listing to 'status=In Process' * Default filter admin users listing to 'status=Active'

* Merge features from main to apcd feature branch * Fix id field --------- Co-authored-by: Garrett Edmonds <43251554+edmondsgarrett@users.noreply.github.com>

…ults-for-query-client

- Removes toggle from success and error message to mimic behavior in other edit modals - Will revisit handling of success messages after React initial release

* react-strap buttons * submit attribute for formik * edit record modal submit attribute --------- Co-authored-by: Chandra Y <cyemparala@tacc.utexas.edu>

* initial commit * Validation Users Edit Record Modal * Exception Modal * ExtensionModal * error above, succsess inline * EditRecord QueryWrapper * QueryWrapper ExceptionModal * pill inline fix * removed non required pills --------- Co-authored-by: Chandra Y <cyemparala@tacc.utexas.edu>

* Fix syntax error in sql * move left join before where

* Created new blank page at URL administration/view-submitter-users * Began building of Submitter Users Page * All Users shown on table; working on filtering by Submitter-User role * Established SQL query to get submitter users; set up views.py * Still working on getting the site to load the data * Got table of submitter users to appear * View modal opens and closes * Edit Modal opens and closes; can't edit records yet * Edit modal updates user names and emails correctly * Added View Submitter Users site to navbar * Linted client code * Removed changes to apcd_cms/src/client/main.tsx * Removed extraneous HTML files and commented-out portions * Update apcd_cms/src/apps/utils/apcd_database.py Simplify the database connection Co-authored-by: Chandra Y <cyemparala@tacc.utexas.edu> * Update apcd_cms/src/apps/utils/apcd_database.py Sort users by user ID Co-authored-by: Chandra Y <cyemparala@tacc.utexas.edu> * Update apcd_cms/src/apps/utils/apcd_database.py Co-authored-by: Chandra Y <cyemparala@tacc.utexas.edu> --------- Co-authored-by: Jeff McMillen <jm63959@TACCs-MacBook-Pro.local> Co-authored-by: Chandra Y <cyemparala@tacc.utexas.edu>

) Co-authored-by: Chandra Y <cyemparala@tacc.utexas.edu>

…ults-for-query-client

chandra-tacc

Can you remove existing usages of staleTime in
useUserFilters and useSubmissionFilters (and in anyother place in the code).
These defaults should help instead of values at each hook

Changes applied! 055ac54

apcd_cms/src/client/src/hooks/admin/useAdmin.ts

chandra-tacc and others added 30 commits February 27, 2024 14:26

Initial commit - using react with APCD admin registration list page (#…

7c63ff2

…276) * Initial commit - using react with APCD page * remove visualizer * Remove print statements * Renamed shared-components to core-components

Initial commit - using react with APCD admin registration list page (#…

139cc78

…276) * Initial commit - using react with APCD page * remove visualizer * Remove print statements * Renamed shared-components to core-components

Update README.md for APCD

c4936c2

Fix dockerfile for static

d32a189

Initial commit - using react with APCD admin registration list page (#…

df65e25

…276) * Initial commit - using react with APCD page * remove visualizer * Remove print statements * Renamed shared-components to core-components

Update README.md for APCD

28c4e38

Fix dockerfile for static

87ecbc2

Merge branch 'feat/apcd-react-conversion' of github.com:TACC/Core-CMS…

09290c0

…-Custom into feat/apcd-react-conversion

Install nodejs as part of docker image

e6d6616

Docker compose dev adjustments for react

e75e6f5

Merge branch 'main' into feat/apcd-react-conversion

f6ecd5f

Merge branch 'feat/apcd-react-conversion' of github.com:TACC/Core-CMS…

7827d80

…-Custom into feat/apcd-react-conversion

task/WP-443 Convert Admin Submissions to React (#288)

39fd3a1

* apcd admin submissions react page version 1 * moved filter and sort functionality to react * Added View logs modal using reactstrap modal --------- Co-authored-by: Chandra Y <cyemparala@tacc.utexas.edu>

Fix merge issue

f9036d0

[ACPD React Conversion]: Setup vite local and add base template with …

86d0fab

…react (#294) * Setup vite local and template with react content * Update Makefile

Update README.md to reflect new changes

c132a2a

Initial version (#299)

10f73ac

Modal windows not included

Task/WP-429 list registrations: filters, pagination and view modal (#298

c07113e

) * List registrations: filters, pagination and view modal * Add pending edit modal and fix org status

List Admin Extensions React Conversion (#289)

599926c

* first run at the react conversion * Update list_admin_extension.html from merge feat branch * Updated incorrect import --------- Co-authored-by: Chandra Y <cyemparala@tacc.utexas.edu>

Add formik, reactstrap, yup for APCD React forms (#302)

451130f

temp commit

530fa50

Submitter Registration List using React (#307)

92074ee

Merge branch 'feat/apcd-react-conversion' of https://github.com/TACC/…

dbf66f3

…Core-CMS-Custom into feat/apcd-react-conversion

Bug/wp 690 empty values should be None (#310)

6fd112a

* Update RegistrationList and ViewRegistrationModal to display 'None' when year, reg_status, fein, license, or naic is undefined or null --------- Co-authored-by: Chandra Y <cyemparala@tacc.utexas.edu>

Merge branch 'feat/apcd-react-conversion' of https://github.com/TACC/…

e3c8656

…Core-CMS-Custom into feat/apcd-react-conversion

WP-713 - add pagination to admin submissions (#322)

e69568b

Merge branch 'feat/apcd-react-conversion' of https://github.com/TACC/…

998fcae

…Core-CMS-Custom into feat/apcd-react-conversion

task/WP-712-Create-useEntity-Hook (#324)

d022604

Co-authored-by: Chandra Y <cyemparala@tacc.utexas.edu>

chandra-tacc and others added 18 commits December 5, 2024 13:17

Bugs/WP-795: clear selection button uniformity (#388)

ce33aa2

* WP-795: clear options button similarity across all pages * set styles

WP-796 - registration list needs to update on modal edits (#391)

dbd9210

WP-799 fix ticket creation api and handle error message (#387)

1776350

tasl/WP-802 & WP-808: Remaining admin default filters (#394)

27eccc2

* Default filter admin submissions listing to 'status=In Process' * Default filter admin users listing to 'status=Active'

Merge features from main to apcd feature branch (#393)

3eca273

* Merge features from main to apcd feature branch * Fix id field --------- Co-authored-by: Garrett Edmonds <43251554+edmondsgarrett@users.noreply.github.com>

initial defaults

482ffe9

Fix Vite build warning

326aac8

Merge branch 'feat/apcd-react-conversion' into Bug/WP-798--Setup-defa…

b93ba29

…ults-for-query-client

bug/Exception-Modal-Success-Exit-Icon-Missing (#397)

d9633d8

- Removes toggle from success and error message to mimic behavior in other edit modals - Will revisit handling of success messages after React initial release

Task/WP-762--Use-Core-Component-Buttons (#381)

af08cb6

* react-strap buttons * submit attribute for formik * edit record modal submit attribute --------- Co-authored-by: Chandra Y <cyemparala@tacc.utexas.edu>

fix merge issue

8b07f10

Quick Fix: APCD, List registrations for submitter is broken (#398)

733d781

* Fix syntax error in sql * move left join before where

File Submissions title update (#399)

64f288a

Update admin list users title + description (#400)

2f7216e

Restrict valid dates for both exception types to format MM/DD/YYYY (#396

5a54828

) Co-authored-by: Chandra Y <cyemparala@tacc.utexas.edu>

Merge branch 'feat/apcd-react-conversion' into Bug/WP-798--Setup-defa…

4e9cd47

…ults-for-query-client

jalowe13 requested review from chandra-tacc and jmcmillenmusic December 11, 2024 21:31

jalowe13 marked this pull request as ready for review December 11, 2024 21:50

chandra-tacc previously requested changes Dec 16, 2024

View reviewed changes

staleTime removal

055ac54

jalowe13 requested a review from chandra-tacc December 17, 2024 21:04

chandra-tacc reviewed Dec 18, 2024

View reviewed changes

apcd_cms/src/client/src/hooks/admin/useAdmin.ts Outdated Show resolved Hide resolved

config settings removal

3cef550

jalowe13 requested a review from chandra-tacc December 18, 2024 22:33

chandra-tacc approved these changes Dec 18, 2024

View reviewed changes

Base automatically changed from feat/apcd-react-conversion to main January 17, 2025 20:57

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Bug/WP-798: Setup Defaults for Query Client #395

Bug/WP-798: Setup Defaults for Query Client #395

jalowe13 commented Dec 6, 2024 •

edited

Loading

chandra-tacc left a comment

Bug/WP-798: Setup Defaults for Query Client #395

Are you sure you want to change the base?

Bug/WP-798: Setup Defaults for Query Client #395

Conversation

jalowe13 commented Dec 6, 2024 • edited Loading

Overview

Related

Changes

Testing

UI

chandra-tacc left a comment

Choose a reason for hiding this comment

jalowe13 commented Dec 6, 2024 •

edited

Loading