Skip to content

[7.x] [Reporting Management] Consolidate report job warnings and add warning for deprecated types (#106184)#107243

Merged
tsullivan merged 1 commit intoelastic:7.xfrom
tsullivan:backport/7.x/pr-106184
Jul 29, 2021
Merged

[7.x] [Reporting Management] Consolidate report job warnings and add warning for deprecated types (#106184)#107243
tsullivan merged 1 commit intoelastic:7.xfrom
tsullivan:backport/7.x/pr-106184

Conversation

@tsullivan
Copy link
Copy Markdown
Member

Backports the following commits to 7.x:

…g for deprecated types (elastic#106184)

* [Reporting Management] Add warning for deprecated types

* record -> job rename

* restore previous report from archive

* more fn testing

* better testing

* use consts

* add table text test

* nicer archived reports archive

* fix test

* add comment about completed

* remove warning button

* consistent status detail message and callout buttons

* rename the deprecated report

* fix status messages

* update copy

* fix duplicated created_at reports

* fix 18n

* back out text add

* fix fn test snapshot

* [Reporting] fix binding of methods that are called via event handlers

* self review

* spot-fix more untranslated content

* Update x-pack/plugins/reporting/public/lib/job.tsx

Co-authored-by: Michael Dokolin <dokmic@gmail.com>

Co-authored-by: Michael Dokolin <dokmic@gmail.com>
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
@tsullivan tsullivan added the backport This PR is a backport of another PR label Jul 29, 2021
@tsullivan tsullivan enabled auto-merge (squash) July 29, 2021 21:40
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
reporting 60 59 -1

Async chunks

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

id before after diff
reporting 71.2KB 68.4KB -2.8KB

Page load bundle

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

id before after diff
reporting 59.5KB 64.7KB +5.1KB

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

@tsullivan tsullivan merged commit d6abb09 into elastic:7.x Jul 29, 2021
@tsullivan tsullivan deleted the backport/7.x/pr-106184 branch November 1, 2021 16:52
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