-
Notifications
You must be signed in to change notification settings - Fork 2
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
Val -> Prod #4384
Val -> Prod #4384
Conversation
* Update ReportDrawer so admins can't submit for users * Light cleaning Co-authored-by: Nicholas Summers <nick@coforma.io>
* unnest pages in report json * refactor spreadsheet * fix all types and prepare drawer and modal forms' * final type tweaks * update copyAdminDisabledStatusToForms utility method * add more tests for utility methods
* api updates * remove undefined * simplify banner provider api calls * fix banner provider tests
* api updates * remove undefined * simplify dynamodb mocking * simplify banner provider api calls * fix banner provider tests
* remove delete button flash * alternate banner deletion display bug solution (#4173) * for scott * removed overkill banner test Co-authored-by: Brax Excell <braxton@coforma.io>
* EntityDrawerReportPage → DrawerReportPage * DynamicDrawerReportPage → ModalDrawerReportPage
* Add Delete Modal to DynamicFields * Add in spinner * Make it so a user can't fully remove dynamic fields * Check that theres always at least 1 dynamic field * Fix bug where modal wouldn't display if you clicked close and opened again * Add test to confirm delete modal will show multiple times * Simplify setting record for deletion * Garrett Edits * Brax Edit * simplify name mapping * Simplify modal spinner Co-authored-by: Nicholas Summers <nick@coforma.io> Co-authored-by: Garrett Rabian <garrett@coforma.io> Co-authored-by: Garrett <57802560+gmrabian@users.noreply.github.com>
#4198) * unnest pages in report json * refactor spreadsheet * fix all types and prepare drawer and modal forms' * final type tweaks * update copyAdminDisabledStatusToForms utility method * add more tests for utility methods * build add/edit modal; style initial page * solve overwrite issue * build out card with no functionality * DynamicDrawerReportPage tests * EntityDrawerReportPage → DrawerReportPage * DynamicDrawerReportPage → ModalDrawerReportPage * switch from record to entity * change dynamicType to entityType * fix failing test * add AddEditEntityModal tests * add EntityCard tests * nick review feedback * Accessibility test fix * hide text if no entities * fix minor type error * garrett feedback changes Co-authored-by: Nicholas Summers <nick@coforma.io> Co-authored-by: Nick Summers <ntsummers1@gmail.com>
* loading state for hydration * Update Form.tsx * banner dashboard hydration loading state * report pages hydration * wrapped it up * Update ReportPageWrapper.tsx Co-authored-by: Brax Excell <braxton@coforma.io>
* unnest pages in report json * refactor spreadsheet * fix all types and prepare drawer and modal forms' * final type tweaks * update copyAdminDisabledStatusToForms utility method * add more tests for utility methods * build add/edit modal; style initial page * solve overwrite issue * build out card with no functionality * DynamicDrawerReportPage tests * EntityDrawerReportPage → DrawerReportPage * DynamicDrawerReportPage → ModalDrawerReportPage * switch from record to entity * change dynamicType to entityType * fix failing test * add AddEditEntityModal tests * Implement simple deletion of entities * add EntityCard tests * Add tests for DeleteEntityModal * Add EntityCard test * fix bug with adding after deleting measures * nick review feedback * Brax Edit * Fix failing test * Accessibility test fix * Accessibility test fix * hide text if no entities * fix minor type error * Fix test * Add test to make sure if bad data is present it still fails elegently * Add test to make sure if bad data is present it still fails elegently Co-authored-by: Braxton Excell <braxton.excell@gmail.com> Co-authored-by: Nicholas Summers <nick@coforma.io> Co-authored-by: Brax Excell <braxton@coforma.io>
* added spreadsheet widgets * Update ReportPageIntro.tsx * normal layout
* unnest pages in report json * refactor spreadsheet * fix all types and prepare drawer and modal forms' * final type tweaks * update copyAdminDisabledStatusToForms utility method * add more tests for utility methods * build add/edit modal; style initial page * solve overwrite issue * build out card with no functionality * DynamicDrawerReportPage tests * EntityDrawerReportPage → DrawerReportPage * DynamicDrawerReportPage → ModalDrawerReportPage * switch from record to entity * change dynamicType to entityType * fix failing test * add AddEditEntityModal tests * add EntityCard tests * Wiring up edit button * Edit is DONE * Hydration of values in modal * Working on failing tests * Unit test is fixed!!!!!!!!!!!! * coverage * Fix issue with types * feedback * Suggested changes Co-authored-by: Braxton Excell <braxton.excell@gmail.com>
* redline section A * PR review changes * added period
* fix nested field validation bug * little naming change
* unnest pages in report json * refactor spreadsheet * fix all types and prepare drawer and modal forms' * final type tweaks * update copyAdminDisabledStatusToForms utility method * add more tests for utility methods * build add/edit modal; style initial page * solve overwrite issue * build out card with no functionality * DynamicDrawerReportPage tests * EntityDrawerReportPage → DrawerReportPage * DynamicDrawerReportPage → ModalDrawerReportPage * switch from record to entity * change dynamicType to entityType * fix failing test * add AddEditEntityModal tests * add EntityCard tests * nick review feedback * Accessibility test fix * hide text if no entities * wip * wip 2 * fix minor type error * wip * modify submitting status locations * data is submitting * wip for nick * Fix status check on entity cards * Hookup Drawer Details and style it * garrett feedback changes * tweaks * fix failing tests * tweak * tweak * Fix bad typeing in delete entity modal * Add tests for EntityCard * Almost clean up tests * add last needed test * tweak * tweaks * trying to fix these merge conflicts and clean things up * clean up * clean up * style tweak for when there's tons of text * unset selected entity * review feedback * combine button * fix up some tests Co-authored-by: Nicholas Summers <nick@coforma.io> Co-authored-by: Nick Summers <ntsummers1@gmail.com>
* unnest pages in report json * refactor spreadsheet * fix all types and prepare drawer and modal forms' * final type tweaks * update copyAdminDisabledStatusToForms utility method * add more tests for utility methods * build add/edit modal; style initial page * solve overwrite issue * build out card with no functionality * DynamicDrawerReportPage tests * EntityDrawerReportPage → DrawerReportPage * DynamicDrawerReportPage → ModalDrawerReportPage * switch from record to entity * change dynamicType to entityType * fix failing test * add AddEditEntityModal tests * add EntityCard tests * nick review feedback * Accessibility test fix * hide text if no entities * wip * wip 2 * fix minor type error * wip * modify submitting status locations * data is submitting * wip for nick * Fix status check on entity cards * Hookup Drawer Details and style it * garrett feedback changes * tweaks * fix failing tests * tweak * tweak * Fix bad typeing in delete entity modal * Add tests for EntityCard * Almost clean up tests * add last needed test * tweak * tweaks * trying to fix these merge conflicts and clean things up * clean up * clean up * style tweak for when there's tons of text * unset selected entity * get drawer hydrating * review feedback * combine button * fix up some tests Co-authored-by: Nicholas Summers <nick@coforma.io> Co-authored-by: Nick Summers <ntsummers1@gmail.com>
* unnest pages in report json * refactor spreadsheet * fix all types and prepare drawer and modal forms' * final type tweaks * update copyAdminDisabledStatusToForms utility method * add more tests for utility methods * build add/edit modal; style initial page * solve overwrite issue * build out card with no functionality * DynamicDrawerReportPage tests * EntityDrawerReportPage → DrawerReportPage * DynamicDrawerReportPage → ModalDrawerReportPage * switch from record to entity * change dynamicType to entityType * fix failing test * add AddEditEntityModal tests * add EntityCard tests * nick review feedback * Accessibility test fix * hide text if no entities * wip * wip 2 * fix minor type error * wip * modify submitting status locations * data is submitting * wip for nick * Fix status check on entity cards * Hookup Drawer Details and style it * garrett feedback changes * tweaks * fix failing tests * tweak * tweak * Fix bad typeing in delete entity modal * Add tests for EntityCard * Almost clean up tests * add last needed test * tweak * tweaks * trying to fix these merge conflicts and clean things up * clean up * clean up * style tweak for when there's tons of text * unset selected entity * get drawer hydrating * review feedback * combine button * fix up some tests * simplify hydration data moulding Co-authored-by: Braxton Excell <braxton.excell@gmail.com> Co-authored-by: Brax Excell <braxton@coforma.io> Co-authored-by: Nicholas Summers <nick@coforma.io> Co-authored-by: Nick Summers <ntsummers1@gmail.com>
* add skipnav to sidebar * simplify styling
* Remove react-query * Remove react-query dependency * Remove old comment Co-authored-by: Nicholas Summers <nick@coforma.io>
* Section C Redlines * Fixup Section C Redlines * Fixup Section C Redlines * Fix additional redlines requested * Additional redlines requested * Tighten up styling * Remove example styling for reducing section gap Co-authored-by: Nicholas Summers <nick@coforma.io>
* Reduce mqClasses with Scotts godly ampersand insight * Missed test deletion xd * Update services/ui-src/src/components/fields/CheckboxField.tsx Co-authored-by: Brax Excell <braxton@coforma.io> * Update services/ui-src/src/components/menus/Sidebar.tsx Co-authored-by: Brax Excell <braxton@coforma.io> Co-authored-by: Nicholas Summers <nick@coforma.io> Co-authored-by: Brax Excell <braxton@coforma.io>
* one layer of abstraction * nest within entitycard folder * separate by type * add a11y tests for different card shapes * pr feedback * simplify
Co-authored-by: Garrett <57802560+gmrabian@users.noreply.github.com>
* one layer of abstraction * nest within entitycard folder * separate by type * parse out entities util * add a11y tests for different card shapes * abstract choice parsing * oops remove
* wip * ha i win * fix failing test * surprise encore * remove commented line
Co-authored-by: Nicholas Summers <nick@coforma.io> Co-authored-by: Brax Excell <braxton@coforma.io>
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.
Access measures page is being weird for me in val. Is it working for others?
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.
Weirdness was on cognito login. My IDM one worked. Not sure why though it's not a problem
Description
How to test
Changed Dependencies
Code author checklist
Reviewer checklist (two different people)