[Security Solution][Endpoint] Paginate actions log with infinite scroll#102261
[Security Solution][Endpoint] Paginate actions log with infinite scroll#102261ashokaditya merged 32 commits intoelastic:masterfrom ashokaditya:feature/olm-paginate_actions_log_infinite_scroll-1245
Conversation
|
@elasticmachine merge upstream |
|
@elasticmachine merge upstream |
|
@elasticmachine merge upstream |
|
merge conflict between base and head |
x-pack/plugins/security_solution/public/management/pages/endpoint_hosts/store/middleware.ts
Outdated
Show resolved
Hide resolved
x-pack/plugins/security_solution/public/management/pages/endpoint_hosts/store/reducer.ts
Show resolved
Hide resolved
x-pack/plugins/security_solution/public/management/pages/endpoint_hosts/store/reducer.ts
Outdated
Show resolved
Hide resolved
x-pack/plugins/security_solution/public/management/pages/endpoint_hosts/store/reducer.ts
Outdated
Show resolved
Hide resolved
...ion/public/management/pages/endpoint_hosts/view/details/components/endpoint_details_tabs.tsx
Show resolved
Hide resolved
...urity_solution/public/management/pages/endpoint_hosts/view/details/endpoint_activity_log.tsx
Show resolved
Hide resolved
refs /pull/101032/commits/c4e933a9c5954ce249942ca66bab380c1dfa79e2#diff-41a74ad41665921620230a0729728f3bf6e27a6f9dc302fb37b0d2061637c212R81
...urity_solution/public/management/pages/endpoint_hosts/view/details/endpoint_activity_log.tsx
Show resolved
Hide resolved
dasansol92
left a comment
There was a problem hiding this comment.
Just added a question but it LGTM 🔥 . Do you have some check or issue somewhere as a reminder for the infinite scroll component?
I'll create a ticket for refactoring the scroll thingy into a component 😊 |
|
@elasticmachine merge upstream |
…et is well hidden below the footer refs 48e3291
paul-tavares
left a comment
There was a problem hiding this comment.
I did not check it out locally and will try to do that a little later. Left some comments and suggestions. let me know if you have any question.
x-pack/plugins/security_solution/public/management/pages/endpoint_hosts/store/action.ts
Show resolved
Hide resolved
x-pack/plugins/security_solution/public/management/pages/endpoint_hosts/store/middleware.ts
Outdated
Show resolved
Hide resolved
x-pack/plugins/security_solution/public/management/pages/endpoint_hosts/store/middleware.ts
Show resolved
Hide resolved
x-pack/plugins/security_solution/public/management/pages/endpoint_hosts/store/reducer.ts
Show resolved
Hide resolved
...ion/public/management/pages/endpoint_hosts/view/details/components/endpoint_details_tabs.tsx
Show resolved
Hide resolved
...urity_solution/public/management/pages/endpoint_hosts/view/details/endpoint_activity_log.tsx
Outdated
Show resolved
Hide resolved
...urity_solution/public/management/pages/endpoint_hosts/view/details/endpoint_activity_log.tsx
Outdated
Show resolved
Hide resolved
x-pack/plugins/security_solution/public/management/pages/endpoint_hosts/view/index.test.tsx
Outdated
Show resolved
Hide resolved
x-pack/plugins/security_solution/public/management/pages/endpoint_hosts/view/index.test.tsx
Outdated
Show resolved
Hide resolved
x-pack/plugins/security_solution/public/management/pages/endpoint_hosts/view/index.test.tsx
Outdated
Show resolved
Hide resolved
…int_hosts/store/reducer.ts Co-authored-by: Paul Tavares <56442535+paul-tavares@users.noreply.github.com>
…int_hosts/view/index.test.tsx Co-authored-by: Paul Tavares <56442535+paul-tavares@users.noreply.github.com>
paul-tavares
left a comment
There was a problem hiding this comment.
Looks good. Thanks for the changes. We can refactor later to drive selected tab using the show= param
👍
|
@elasticmachine merge upstream |
|
@elasticmachine merge upstream |
…ll (elastic#102261) * Show loading below the list when loading fixes elastic/security-team/issues/1245 * use intersection observer to load data when callout is visible fixes elastic/security-team/issues/1245 * remove unused `total` from API response refs 4f7d18b * toggle ability to paging based on API response and target intersection fixes elastic/security-team/issues/1245 * use a invisible target * display a message when end of log fixes elastic/security-team/issues/1245 * remove search bar fixes elastic/security-team/issues/1245 * refresh data fixes elastic/security-team/issues/1245 * rename refs 85e5add * add refresh button to empty state * add translations for copy * remove refresh button * load activity log for endpoint on activity log tab selection fixes elastic/security-team/issues/1312 * reset paging correctly on activity log tab selection * fix variable mixup refs elastic/pull/101032/commits/c4e933a9c5954ce249942ca66bab380c1dfa79e2#diff-41a74ad41665921620230a0729728f3bf6e27a6f9dc302fb37b0d2061637c212R81 * fix react warning refs 697a3c3 * clean up review changes * use the complicated flyout version instead of styled version refs https://elastic.github.io/eui/#/layout/flyout#more-complicated-flyout refs https://github.com/elastic/kibana/pull/99795/files#r635810660 refs c26a7d4 * Page only when scrolled (so that info message is shown after paging once) fixes elastic/security-team#1245 (comment) * add tests fixes elastic/security-team/issues/1312 fixes elastic/security-team/issues/1245 * increase the parent container's height to ensure that the scroll target is well hidden below the footer refs 48e3291 * Update x-pack/plugins/security_solution/public/management/pages/endpoint_hosts/store/reducer.ts Co-authored-by: Paul Tavares <56442535+paul-tavares@users.noreply.github.com> * Update x-pack/plugins/security_solution/public/management/pages/endpoint_hosts/view/index.test.tsx Co-authored-by: Paul Tavares <56442535+paul-tavares@users.noreply.github.com> * address review changes * cleanup callback and effect Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com> Co-authored-by: Paul Tavares <56442535+paul-tavares@users.noreply.github.com>
💚 Backport successful
This backport PR will be merged automatically after passing CI. |
…ll (#102261) (#103047) * Show loading below the list when loading fixes elastic/security-team/issues/1245 * use intersection observer to load data when callout is visible fixes elastic/security-team/issues/1245 * remove unused `total` from API response refs 4f7d18b * toggle ability to paging based on API response and target intersection fixes elastic/security-team/issues/1245 * use a invisible target * display a message when end of log fixes elastic/security-team/issues/1245 * remove search bar fixes elastic/security-team/issues/1245 * refresh data fixes elastic/security-team/issues/1245 * rename refs 85e5add * add refresh button to empty state * add translations for copy * remove refresh button * load activity log for endpoint on activity log tab selection fixes elastic/security-team/issues/1312 * reset paging correctly on activity log tab selection * fix variable mixup refs /pull/101032/commits/c4e933a9c5954ce249942ca66bab380c1dfa79e2#diff-41a74ad41665921620230a0729728f3bf6e27a6f9dc302fb37b0d2061637c212R81 * fix react warning refs 697a3c3 * clean up review changes * use the complicated flyout version instead of styled version refs https://elastic.github.io/eui/#/layout/flyout#more-complicated-flyout refs https://github.com/elastic/kibana/pull/99795/files#r635810660 refs c26a7d4 * Page only when scrolled (so that info message is shown after paging once) fixes elastic/security-team#1245 (comment) * add tests fixes elastic/security-team/issues/1312 fixes elastic/security-team/issues/1245 * increase the parent container's height to ensure that the scroll target is well hidden below the footer refs 48e3291 * Update x-pack/plugins/security_solution/public/management/pages/endpoint_hosts/store/reducer.ts Co-authored-by: Paul Tavares <56442535+paul-tavares@users.noreply.github.com> * Update x-pack/plugins/security_solution/public/management/pages/endpoint_hosts/view/index.test.tsx Co-authored-by: Paul Tavares <56442535+paul-tavares@users.noreply.github.com> * address review changes * cleanup callback and effect Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com> Co-authored-by: Paul Tavares <56442535+paul-tavares@users.noreply.github.com> Co-authored-by: Ashokaditya <am.struktr@gmail.com> Co-authored-by: Paul Tavares <56442535+paul-tavares@users.noreply.github.com>
💔 Build Failed
Failed CI StepsTest FailuresKibana Pipeline / general / X-Pack API Integration Tests.x-pack/test/api_integration/apis/management/index_lifecycle_management/policies·js.apis management index lifecycle management policies list should have a default policy to manage the Watcher history indicesStandard OutStack TraceKibana Pipeline / general / X-Pack API Integration Tests.x-pack/test/api_integration/apis/management/index_lifecycle_management/policies·js.apis management index lifecycle management policies list should have a default policy to manage the Watcher history indicesStandard OutStack TraceMetrics [docs]Module Count
Async chunks
History
To update your PR or re-run it, just comment with: |
…er (#110233) * account for API errors and uninitialized state before fetching data fixes /issues/107129 * better name refs /pull/102261 * don't show date picker when loading data initially fixes /issues/107129 * use a readable selector instead review changes * remove redundant data fetch using paging action on tab switch. refs /pull/102261 * remove redundant validation review comments Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
…er (elastic#110233) * account for API errors and uninitialized state before fetching data fixes elastic/issues/107129 * better name refs elastic/pull/102261 * don't show date picker when loading data initially fixes elastic/issues/107129 * use a readable selector instead review changes * remove redundant data fetch using paging action on tab switch. refs elastic/pull/102261 * remove redundant validation review comments Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
…er (elastic#110233) * account for API errors and uninitialized state before fetching data fixes elastic/issues/107129 * better name refs elastic/pull/102261 * don't show date picker when loading data initially fixes elastic/issues/107129 * use a readable selector instead review changes * remove redundant data fetch using paging action on tab switch. refs elastic/pull/102261 * remove redundant validation review comments Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
…er (#110233) (#110526) * account for API errors and uninitialized state before fetching data fixes /issues/107129 * better name refs /pull/102261 * don't show date picker when loading data initially fixes /issues/107129 * use a readable selector instead review changes * remove redundant data fetch using paging action on tab switch. refs /pull/102261 * remove redundant validation review comments Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com> Co-authored-by: Ashokaditya <am.struktr@gmail.com>
…er (#110233) (#110525) * account for API errors and uninitialized state before fetching data fixes /issues/107129 * better name refs /pull/102261 * don't show date picker when loading data initially fixes /issues/107129 * use a readable selector instead review changes * remove redundant data fetch using paging action on tab switch. refs /pull/102261 * remove redundant validation review comments Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com> Co-authored-by: Ashokaditya <am.struktr@gmail.com>
Summary
This PR builds on top of /pull/101032 and /pull/99795 and enables users to paginate data by scrolling to the end of the list instead of clicking on a button.
screenshots

empty state

Checklist
Delete any items that are not applicable to this PR.