-
Notifications
You must be signed in to change notification settings - Fork 8
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
TTA overview widget (#316) #479
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* First pass at a pattern for widgets * Set coverage limits to 0 so we deploy to staging These limits will be increased back to what they are on main before merging. * Reset coverage limits * TTA Overview widget initial commit * Add backend tests * Deploy to sandbox * Fix lint error * Fix frontend lint errors * Merge main and fix invalid region * Refactor test * Add frontend tests * Add frontend tests * Styling improvements * Add region number in caption * Update mocks in landing tests * Adjusted sizing, flickering * Add temp border to overview widget Co-authored-by: Josh Salisbury <[email protected]>
kryswisnaskas
requested review from
OHS-christine-nguyen,
PatricePascual-ACF and
thewatermethod
June 15, 2021 20:58
PatricePascual-ACF
approved these changes
Jun 15, 2021
thewatermethod
added a commit
that referenced
this pull request
Nov 15, 2021
* fix prop type * db.sequelize.close returns a promise * fix unit test * building backend route for grantee search * create query for search * clean up function * add unit test * add size limit to query * fix broken ui test * update ui test for new features * add another ui test * beefed up unit test * beefed up second unit test * refactor fetching of widget data * add granteeId to scopes * remove unused file * remove widgets example page * add unit test * fix broken test * back end for grants list widget * build out GranteeResults * fixed placement of statistics to match mockup * add grantee results table * write ui test * add additional ui tests * backend fixes for grants program types * backend unit tests * update yarn vuln * starting to fill test coverage gaps * more work on ui tests * return fetchers to prev state * ui test for grantee search * another ui test * testing linter * remove test * test * remove inline function * fix bug where sort wasn't firing * why not? * removed shot in the dark * test this lint * test this lint * on apply remove as a test * defeated, we add no an eslint ignore * updates wip * fix unit tests * linter fixes * changes for grantee to use scopes with fixes * finish UI tests * more grantee list updates * first version of grants list widget * updates * unit test for Grants List FE * fix front end tests * fix all backend tests * minor ui update * change ids to see if test passes * updates * fix ui * add feature flag * clean up feature flag component and some naming * clean up fetcher * clean up css * rename determineFiltersToScopes to filtersToScopes * cleanup feature flag usage * filtering via model type * leverage existing filter code * more tweaking of feature flag * add missing param * add default sorting to grantee service * update grantee service * write unit tests for new scopes * fix ui tests * fix lint error * fix backend unit tests * fix lint errors * feature flag grantee record * refactor search endpoint to use new scopes * fix issues with grantee record page * fix flaky frontend test and iron out bugs with merge * remove extraneous comment * working on proper error handling; * remove failing tests for now * add back in unit tests * simplify fetching data * improve backend coverage * no magic numbers * add new tab components; * new layout changes * sensible defaults * fix backend test * wrote "a lot" of tests * add to feature flag tests * switch url structure * remove unneeded error override prop * fix indentation * users can't set 'modelType' * hoist Filter to its own scope * rename granteebyScopes to granteeById * refactor granteeById to use program table * rename granteesByNameAndRegion * required false on included model * remove unneeded filter * refactor loading logic for grantee search form * remove unnecessary condition * refactor programs on frontend * fix some unit tests * fix unit tests * remove console logs * create basic filter component * rearranging componentry * putting it all together * add todo comment * flopped those bad boys * flopped menu item order * add configurable fields to overview widget * always show search results (simplify loading logic) * show side nav as active when in any part of grantee record flow * style changes * refactor tooltips, add them to the overview widget * fix back to search link * fix grantee results page * fix tooltip text wrap, screenreader on overview * fix UI tests for grantee record page * simplify filter management * clean up filters some more * further filter cleanup * allow year to date as a filter option; * add key to render, refactor components * role filter should show overlap * working out the kinks * fix filter state and match mockup * add keyboard handling and descriptive page titles * further styling * writing tests * start unit tests * trying to fix post merge mistakes * fixing some merge mistakes * add some UI test * add additional unit tests * fix botched backend test and add frontend test * some small accessibility improvements * add refactor plan * begin refactoring * refactor and seperate button select and date range select * start another refactor * refactor menu usage * refactor dropdowns to use one component * add test for grantee search * cleanup TTA History filter * convert filter menu to dropdown menu * ui fixes after refactoring * fix more ui bugs * fill in some test coverage * add filter menu item test * get rid of duplicate constant * add test for filter date range * more test coverage * fix bug with filter blurring * fix blur interactions * fix weird checkbox behavior * bring date icons inline; * reset condition on topic change * remove debugging comment * ditch unused code so we don\'t have to test it * adapt test since query is killed by topic change * write test for button select * remove phantom state from checkbox select * refactor formatDateRange function * add test to date range select * remove unused code from filter menu that we shouldnt test * fix date range select label bug * fix specialist select blur bug * [TTAHUB-404] Add target populations to Grantee record page (#480) * add label to input in search form * create new widget front and back * ui test * put it in a grid * fix ui test * clean up tests * radd scopes to goals for grant * I put the where in the wrong place (oopsie) * fix test * Add SES to the diagram * Add to ignored yarn vulnerabilities Co-authored-by: unknown <[email protected]> * fix weird blur behavior * make grids agree * resolve merge issues * TTA-HUB: 397 Add Pill Filters to Grantee Page (#481) * first version of filter pills * added first version of filter pills * added tests for filter pill ui * change colors and add test * fixed issues found by Matt * fixed filter issues and specialists check boxes * fixed pill row spacing and moved AND to front * fix audit vuln * fix lint errors * wip * css fixes * code clean up * added unit test for tool tip and fixed filter pill test * align text in ellipsis pill * fixes based on Joshs PR review Co-authored-by: Matt Bevilacqua <[email protected]> * neaten the prop types * fighting against extra api calls * clear out query on condition * fix ui tests * add column header test * remove unneeded code * add another ui test * pass correct filters to the frequency graph * fix weird useState * refactor filter usage across TTA History * apply filters to table as well * add min width to menu * remove all specialists as an option * fix table colors * fix grantee record ui test * remove toggle all checked ui test * fix other failing tests * add test to column header * test * no div + caption * cleanup test * add test to mock * lwrite a test that works * we have to stop meeting like this Co-authored-by: Adam Levin <[email protected]> Co-authored-by: Adam Levin <[email protected]> Co-authored-by: Josh Salisbury <[email protected]> Co-authored-by: unknown <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of change
This PR builds on #300 adding support for a TTA overview widget on the landing page.
How to test
Note * - outstanding issues - currently the dropdown menu won't close on blur and the region selector needs a keyboard navigation:
https://ocio-jira.acf.hhs.gov/browse/TTAHUB-197
https://ocio-jira.acf.hhs.gov/browse/TTAHUB-198
Issue(s)
Checklists
Every PR
Production Deploy
After merge/deploy