Skip to content

[7.15] [RAC] EuiDataGrid pagination (#109269)#110316

Merged
kibanamachine merged 1 commit intoelastic:7.15from
kibanamachine:backport/7.15/pr-109269
Aug 26, 2021
Merged

[7.15] [RAC] EuiDataGrid pagination (#109269)#110316
kibanamachine merged 1 commit intoelastic:7.15from
kibanamachine:backport/7.15/pr-109269

Conversation

@kibanamachine
Copy link
Contributor

Backports the following commits to 7.15:

* Update T-Grid to use DataGrid pagination

* It also improves the Gtid loading state
* DataGrid pagination makes sure that we display the grid with the proper height.

* Add DataGrid height hack to t-grid

HUGE HACK!!!

DataGrtid height isn't properly calculated when the grid has horizontal scroll.
elastic/eui#5030

In order to get around this bug we are calculating `DataGrid` height here and setting it as a prop.

Please revert this commit  and allow DataGrid to calculate its height when the bug is fixed.

* Apply DataGrid laoding and pagination changes to observability

* Fix cypress tests

* Fix t-grid page render bug on Observability

* some pagination fixes

* hide table when analyzer active

* isolate exported function

Co-authored-by: semd <sergi.massaneda@elastic.co>
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Aug 26, 2021
@kibanamachine kibanamachine enabled auto-merge (squash) August 26, 2021 18:44
@kibanamachine
Copy link
Contributor Author

💛 Build succeeded, but was flaky


Test Failures

Kibana Pipeline / general / "before all" hook for "should not allow user with read only privileges to attach alerts to cases".Alerts timeline Privileges: read only "before all" hook for "should not allow user with read only privileges to attach alerts to cases"

Link to Jenkins

Stack Trace

Failed Tests Reporter:
  - Test has not failed recently on tracked branches

Error: Timed out while retrying, last result was: {false}

Because this error occurred during a `before all` hook we are skipping the remaining tests in the current suite: `Alerts timeline`

Although you have test retries enabled, we do not retry tests when `before all` or `after all` hooks fail
    at Context.completeOrRetry (http://localhost:61201/__cypress/tests?p=cypress/support/index.js:380:13)

Metrics [docs]

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
timelines 843 845 +2

Async chunks

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

id before after diff
observability 493.1KB 493.2KB +141.0B
securitySolution 6.6MB 6.6MB +885.0B
timelines 444.5KB 436.9KB -7.6KB
total -6.6KB

Page load bundle

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

id before after diff
timelines 307.4KB 307.8KB +496.0B
Unknown metric groups

API count

id before after diff
timelines 963 966 +3

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

cc @machadoum

@kibanamachine kibanamachine merged commit 76e92d8 into elastic:7.15 Aug 26, 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