Skip to content

[7.x] [Logs UI] Refactor log position to hooks (#53540)#54204

Merged
Zacqary merged 1 commit intoelastic:7.xfrom
Zacqary:backport/7.x/pr-53540
Jan 8, 2020
Merged

[7.x] [Logs UI] Refactor log position to hooks (#53540)#54204
Zacqary merged 1 commit intoelastic:7.xfrom
Zacqary:backport/7.x/pr-53540

Conversation

@Zacqary
Copy link
Copy Markdown
Contributor

@Zacqary Zacqary commented Jan 7, 2020

Backports the following commits to 7.x:

* Move URL state to hook

* Fix log filter URL state infinite loop

* Initial refactor of log position to hooks

* Simplify and reimplement controlsShouldDisplayTargetPosition

* Fix live streaming

* Flatten logposition destructuring

* Revert "Move URL state to hook"

This reverts commit 4e04aa0.

# Conflicts:
#	x-pack/legacy/plugins/infra/public/containers/logs/log_filter/use_log_filter_url_state.tsx
#	x-pack/legacy/plugins/infra/public/pages/logs/stream/page_providers.tsx

* Fix unused imports

* Fix link-to test

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
@Zacqary Zacqary added the backport This PR is a backport of another PR label Jan 7, 2020
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

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

@Zacqary Zacqary merged commit 39050dd into elastic:7.x Jan 8, 2020
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