Skip to content

[7.x] [Reporting] Clean Up TypeScript Definitions (#76566)#78210

Merged
tsullivan merged 2 commits intoelastic:7.xfrom
tsullivan:backport/7.x/pr-76566
Sep 23, 2020
Merged

[7.x] [Reporting] Clean Up TypeScript Definitions (#76566)#78210
tsullivan merged 2 commits intoelastic:7.xfrom
tsullivan:backport/7.x/pr-76566

Conversation

@tsullivan
Copy link
Copy Markdown
Member

Backports the following commits to 7.x:

@tsullivan tsullivan added the backport This PR is a backport of another PR label Sep 22, 2020
@tsullivan tsullivan force-pushed the backport/7.x/pr-76566 branch from 54bc934 to f7e14b8 Compare September 22, 2020 20:26
* [Reporting] Simplify Export Type Definitions, use defaults for generics, refactor

* ReportApiJSON interface for common

* rename JobSummary to JobStatusBucket for clarity

* revert unneeded create mock changes

* clean up the diff

* revert changes to worker.js

* rewrite comment

* rename type to jobtype in JobStatusBucket

* allow type inference

* JobSummarySet

* remove odd comment

* Reflect that browser timezone may be undefined in the BaseParams

* comment about optional browserTimezone

* revert unecessary es archive change

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
@tsullivan tsullivan force-pushed the backport/7.x/pr-76566 branch from f7e14b8 to e35b516 Compare September 22, 2020 20:27
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Build metrics

page load bundle size

id value diff baseline
reporting 304.3KB +171.0B 304.2KB

distributable file count

id value diff baseline
default 47662 +1 47661

History

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

@tsullivan tsullivan merged commit 6e37033 into elastic:7.x Sep 23, 2020
@tsullivan tsullivan deleted the backport/7.x/pr-76566 branch September 23, 2020 00:08
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