Skip to content

Commit

Permalink
[Backport 2.16] [Bug] Remove ag grid package (#2001) (#2003)
Browse files Browse the repository at this point in the history 
* [Bug] Remove ag grid package (#2001)

* remove ag grid

Signed-off-by: Adam Tackett <tackadam@amazon.com>

* update yarn lock

Signed-off-by: Adam Tackett <tackadam@amazon.com>

* update release notes

Signed-off-by: Adam Tackett <tackadam@amazon.com>

---------

Signed-off-by: Adam Tackett <tackadam@amazon.com>
Co-authored-by: Adam Tackett <tackadam@amazon.com>
(cherry picked from commit 805c54f)

* updated snapshots

Signed-off-by: Shenoy Pratik <sgguruda@amazon.com>

---------

Signed-off-by: Shenoy Pratik <sgguruda@amazon.com>
Co-authored-by: Adam Tackett <105462877+TackAdam@users.noreply.github.com>
  • Loading branch information
@ps48 @TackAdam
ps48 and TackAdam authored Jul 29, 2024
1 parent c220905 commit e31ba87
Show file tree
Hide file tree
Showing 13 changed files with 14 additions and 432 deletions.
2 changes: 0 additions & 2 deletions package.json
View file
Original file line number Diff line number Diff line change
Expand Up @@ -16,13 +16,11 @@
"cypress:parallel": "cypress-parallel -s cypress:run -t 2 -d .cypress/integration"
},
"dependencies": {
"@ag-grid-community/styles": "^31.3.4",
"@algolia/autocomplete-core": "^1.4.1",
"@algolia/autocomplete-theme-classic": "^1.2.1",
"@nteract/outputs": "^3.0.11",
"@nteract/presentational-components": "^3.4.3",
"@reduxjs/toolkit": "^1.6.1",
"ag-grid-react": "^31.3.4",
"ajv": "^8.11.0",
"antlr4": "4.8.0",
"antlr4ts": "^0.5.0-alpha.4",
Expand Down
2 changes: 1 addition & 1 deletion public/components/common/search/__tests__/__snapshots__/search.test.tsx.snap
View file
Original file line number Diff line number Diff line change
Expand Up @@ -571,7 +571,7 @@ exports[`Explorer Search component renders basic component 1`] = `
hasArrow={true}
isOpen={false}
ownFocus={true}
panelPaddingSize="m"
panelPaddingSize="s"
>
<div
className="euiPopover euiPopover--anchorDownLeft"
Expand Down
4 changes: 2 additions & 2 deletions public/components/custom_panels/__tests__/__snapshots__/custom_panel_view.test.tsx.snap
View file
Original file line number Diff line number Diff line change
Expand Up @@ -2153,7 +2153,7 @@ exports[`Panels View Component renders panel view container with visualizations
hasArrow={true}
isOpen={false}
ownFocus={true}
panelPaddingSize="m"
panelPaddingSize="s"
>
<div
className="euiPopover euiPopover--anchorDownLeft"
Expand Down Expand Up @@ -4404,7 +4404,7 @@ exports[`Panels View Component renders panel view container without visualizatio
hasArrow={true}
isOpen={false}
ownFocus={true}
panelPaddingSize="m"
panelPaddingSize="s"
>
<div
className="euiPopover euiPopover--anchorDownLeft"
Expand Down
15 changes: 0 additions & 15 deletions ...ic/components/event_analytics/explorer/visualizations/workspace_panel/workspace_panel.tsx
View file
Original file line number Diff line number Diff line change
Expand Up @@ -26,21 +26,6 @@ export function WorkspacePanel({ visualizations }: IWorkSpacePanel) {

return (
<div className="explorerViz__commonPanel ws__workspace_visPanel">
<EuiPanel className="workspace_visPanel" paddingSize="s" color="plain" hasBorder={false}>
<EuiSwitch
className="workspace_visPanel--switch"
label="Table view"
type="button"
disabled={isEmpty(visualizations?.data?.explorer?.explorerData)}
checked={isTableViewOn}
onChange={() => {
setIsTableViewOn((staleState) => !staleState);
}}
aria-describedby="table view switcher"
data-test-subj="workspace__dataTableViewSwitch"
compressed
/>
</EuiPanel>
<EuiPanel
paddingSize="s"
className={`ws__workspace_visPanel--space ${
Expand Down
2 changes: 1 addition & 1 deletion public/components/metrics/top_menu/__tests__/__snapshots__/top_menu.test.tsx.snap
View file
Original file line number Diff line number Diff line change
Expand Up @@ -516,7 +516,7 @@ exports[`Metrics Top Menu Component renders Top Menu Component when enabled 1`]
hasArrow={true}
isOpen={false}
ownFocus={true}
panelPaddingSize="m"
panelPaddingSize="s"
>
<div
className="euiPopover euiPopover--anchorDownLeft"
Expand Down
4 changes: 2 additions & 2 deletions ...onents/trace_analytics/components/common/__tests__/__snapshots__/search_bar.test.tsx.snap
View file
Original file line number Diff line number Diff line change
Expand Up @@ -220,7 +220,7 @@ exports[`Search bar components renders date picker 1`] = `
hasArrow={true}
isOpen={false}
ownFocus={true}
panelPaddingSize="m"
panelPaddingSize="s"
>
<div
className="euiPopover euiPopover--anchorDownLeft"
Expand Down Expand Up @@ -729,7 +729,7 @@ exports[`Search bar components renders search bar 1`] = `
hasArrow={true}
isOpen={false}
ownFocus={true}
panelPaddingSize="m"
panelPaddingSize="s"
>
<div
className="euiPopover euiPopover--anchorDownLeft"
Expand Down
6 changes: 3 additions & 3 deletions ...onents/trace_analytics/components/services/__tests__/__snapshots__/services.test.tsx.snap
View file
Original file line number Diff line number Diff line change
Expand Up @@ -938,7 +938,7 @@ exports[`Services component renders empty services page 1`] = `
hasArrow={true}
isOpen={false}
ownFocus={true}
panelPaddingSize="m"
panelPaddingSize="s"
>
<div
className="euiPopover euiPopover--anchorDownLeft"
Expand Down Expand Up @@ -3145,7 +3145,7 @@ exports[`Services component renders jaeger services page 1`] = `
hasArrow={true}
isOpen={false}
ownFocus={true}
panelPaddingSize="m"
panelPaddingSize="s"
>
<div
className="euiPopover euiPopover--anchorDownLeft"
Expand Down Expand Up @@ -4783,7 +4783,7 @@ exports[`Services component renders services page 1`] = `
hasArrow={true}
isOpen={false}
ownFocus={true}
panelPaddingSize="m"
panelPaddingSize="s"
>
<div
className="euiPopover euiPopover--anchorDownLeft"
Expand Down
6 changes: 3 additions & 3 deletions ...components/trace_analytics/components/traces/__tests__/__snapshots__/traces.test.tsx.snap
View file
Original file line number Diff line number Diff line change
Expand Up @@ -956,7 +956,7 @@ exports[`Traces component renders empty traces page 1`] = `
hasArrow={true}
isOpen={false}
ownFocus={true}
panelPaddingSize="m"
panelPaddingSize="s"
>
<div
className="euiPopover euiPopover--anchorDownLeft"
Expand Down Expand Up @@ -2634,7 +2634,7 @@ exports[`Traces component renders jaeger traces page 1`] = `
hasArrow={true}
isOpen={false}
ownFocus={true}
panelPaddingSize="m"
panelPaddingSize="s"
>
<div
className="euiPopover euiPopover--anchorDownLeft"
Expand Down Expand Up @@ -4315,7 +4315,7 @@ exports[`Traces component renders traces page 1`] = `
hasArrow={true}
isOpen={false}
ownFocus={true}
panelPaddingSize="m"
panelPaddingSize="s"
>
<div
className="euiPopover euiPopover--anchorDownLeft"
Expand Down
Loading

0 comments on commit e31ba87

Please sign in to comment.