Collapsible checks catalog #1369
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
nelsonkopliku
force-pushed
the
collapsible-checks-catalog
branch
from
6e6e3ae to
6c6b610
Compare
nelsonkopliku
added
enhancement
nelsonkopliku
force-pushed
the
collapsible-checks-catalog
branch
from
14863cb to
67dfa27
Compare
nelsonkopliku
force-pushed
the
new-checks-ui
branch
from
9b3775a to
40e7a70
Compare
nelsonkopliku
force-pushed
the
collapsible-checks-catalog
branch
from
67dfa27 to
effda8d
Compare
EMaksy
reviewed
Apr 27, 2023
dottorblaster
requested changes
Apr 28, 2023
EMaksy
reviewed
Apr 28, 2023
nelsonkopliku
force-pushed
the
collapsible-checks-catalog
branch
from
f8ad65f to
a2c975d
Compare
EMaksy
requested changes
Apr 28, 2023
nelsonkopliku
force-pushed
the
collapsible-checks-catalog
branch
from
a2c975d to
504eb58
Compare
nelsonkopliku
force-pushed
the
collapsible-checks-catalog
branch
from
504eb58 to
3c54166
Compare
EMaksy
approved these changes
Apr 28, 2023
nelsonkopliku
added a commit
that referenced
this pull request
May 4, 2023
* Add an Accordion Component * Refactor ChecksCatalog to have collapsible groups * Adjust cypress test to select correct item in the DOM * Refactor CheckItem to use the new Accordion component * Refactor ChecksSelectionGroup to use the new Accordion component * Adjust z-index for ProviderSelection
nelsonkopliku
added a commit
that referenced
this pull request
May 4, 2023
* Add an Accordion Component * Refactor ChecksCatalog to have collapsible groups * Adjust cypress test to select correct item in the DOM * Refactor CheckItem to use the new Accordion component * Refactor ChecksSelectionGroup to use the new Accordion component * Adjust z-index for ProviderSelection
nelsonkopliku
added a commit
that referenced
this pull request
May 4, 2023
* Check based execution results (#1223) * Add an ExecutionHeader component * Add a selector to get hostnames of a given cluster * Refactor ExecutionResults to a Check based visualization * Object tree component (#1228) * Fix storybook configuration with @static alias * Add object tree factory * Add ObjectTree React component * Add tests * Add react-accessible-treeview and a webpack plugin required for Storybook * Add desctructuredArrayIgnorePattern option to eslint * Address review comments * Add WIP * Re-add checks filtering (#1231) * Remove debug code (#1235) * Improve error handling on checks results page (#1232) * Enhance NotificationBox to support straight message text or a list of messages to show * Improve error handling on checks results page * Keep a loading box while the execution is running * Fix ExecutionHeader test with new warning text * Open remediation modal when check id is clicked in the ExecutionResults view (#1256) * Open remediation modal when checkId is clicked * Fix determining expectations results (#1246) * Add a getCheckExpectations utility to support check based view * Improve CheckResultOutline to properly handle more scenarios * Adjust ExecutionResults component so that it provides catalog expectations to a Check result outline * Address review comments * Adjust styling * Check result detail pt1 (#1248) * Chore: Use getClusterHostNames in ExecutionResultsPage * Make TargetResult clickable * Navigate to CheckDetail page when clicking on a target result row * Add an ExecutionContainer component that wraps loading states * Add a CheckResultInfoBox component that gives information about a check's execution context * Add a CheckDetailHeader that provides the introductory section of a check detail page * Add an initial version of the CheckResultDetail * Add a CheckResultDetailPage as a container interacting with the state * Wire up the newly created CheckResultDetailPage to the app router * Improve component testing * Improve selection for ExecutionResultsPage and CheckResultDetailPage * Rename getLastExecutionContext selector to getLastExecutionData * Improve loading effects * Improve cluster hosts retrieval for execution results/check detail (#1268) * Chore: cleanup CheckResultInfoBox classname * Add a getClusterHosts to get all the hosts of a cluster * Use getClusterHosts in getLastExecutionData selector * Adjust test factories in relation to hosts * Refactor ExecutionResults to use the provided cluster hosts * Refactor CheckResultDetailPage to use provided cluster hosts and pass those to the check detail * Add empty state subcomponent for the table component (#1279) * Add empty state component and test * Check Result Detail pt2 (#1264) * Chore: extract and test transformation functions from CheckResultOutline * Implement a Check detail for a Host target * Determine the target id based on target type in CheckResultDetailPage * Split CheckResultDetail subcomponents in dedicated files * Address review feedbacks * Improve naming * Visually identify premium checks in the ExecutionResults view (#1293) * Add PremiumPill to ExecutionResults component * Cluster wide Check Result Detail (#1291) * Add an HealthIcon to CheckDetailHeader * Centralize enrichment of execution data agents check results with hostnames into selector * Cleanup extraction of expect_same statements results * Support both host and cluster wide ExpectationsResults * Add the ability to extract facts for cluster wide checks * Add support for both host and cluster wide facts to GatheredFacts component * Make CheckResultDetail support host and cluster wide checks * Address review feedbacks * Fact value primitive types (#1317) * Display raw values for primitive types * Add FactValue tests and detail stories * Fix factories using transient params * Fix props in CheckResultDetail test * Add some margin between cluster-wide facts (#1336) * Clean up legacy code (#1325) * Catalog provider filter (#1363) * Move provider filter to the right in catalog view * Add ProviderLabel storybook story * Update providerSelection and add story * Use the update ProviderSelection in catalog * Handle All entry in provider selection * Remove unwanted comments * Use all providers in ProviderSelection story * Refacto buildUpdateCatalogAction function * Add url error handling for the CheckResultDetailPage (#1364) * Refactor NotFound component, add test and story * Add url error handling for CheckResultDetailPage * Collapsible checks catalog (#1369) * Add an Accordion Component * Refactor ChecksCatalog to have collapsible groups * Adjust cypress test to select correct item in the DOM * Refactor CheckItem to use the new Accordion component * Refactor ChecksSelectionGroup to use the new Accordion component * Adjust z-index for ProviderSelection * Warning banner improvement (#1371) * Refactor providerWarningBanners to also support result specific banners and expose retrieving functions * Use the banner retrieving functions * Refactor warning banners usage --------- Co-authored-by: Alessio Biancalana <alessio.biancalana@suse.com> Co-authored-by: EMaksy <emaksymenko@suse.com> Co-authored-by: Xabier Arbulu Insausti <xarbulu@suse.com>
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
This PR
Accordioncomponent wrapping headlessuiDisclosureHow was this tested?
Added tests and storybook for the Accordion. Adjusted existing tests where needed.