Skip to content

[7.x] [Fleet] Remove duplication between two files #103282 (#103282)#103329

Merged
kibanamachine merged 1 commit intoelastic:7.xfrom
kibanamachine:backport/7.x/pr-103282
Jun 24, 2021
Merged

[7.x] [Fleet] Remove duplication between two files #103282 (#103282)#103329
kibanamachine merged 1 commit intoelastic:7.xfrom
kibanamachine:backport/7.x/pr-103282

Conversation

@kibanamachine
Copy link
Contributor

Backports the following commits to 7.x:

## Summary

`public/applications/integrations/constants.tsx` and
`public/applications/integrations/sections/epm/constants.tsx` are identical except for this line in `public/applications/integrations/constants.tsx`

```ts
export * from '../../constants';
```

This PR removes all the duplication from the "upper" file (`public/applications/integrations/constants.tsx`) and leaves the other code "down" in `/sections/epm/` closer to where it's used.

Initially, I deleted `public/applications/integrations/constants.tsx` entirely but several files do `import` the constants it exports, so I left it.
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Jun 24, 2021
@kibanamachine kibanamachine enabled auto-merge (squash) June 24, 2021 16:59
@kibanamachine
Copy link
Contributor Author

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
fleet 692.6KB 692.6KB -2.0B

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
fleet 441.4KB 440.7KB -682.0B

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @jfsiii

@kibanamachine kibanamachine merged commit d2196c6 into elastic:7.x Jun 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants