Skip to content

Conversation

@afgomez
Copy link
Contributor

@afgomez afgomez commented Jan 6, 2020

Backports the following commits to 7.x:

* Scaffold `log_entries/entries` route

* Scaffold a log entry response

* Add `after` pagination

* Add `before` pagination

* Process `query` parameter

* Use pre-existing structure for the columns

* Change type of date ranges

We will move the responsibility to parse the dates to the client. The
API will only take timestamps

* Add `center` parameter

Allows consumers of the API to get log items around a certain cursor

* Change default page size

* Test the defaults of the API

* Add optional `size` parameter

This makes easier to test the pagination. By default it returns a 200
size page.

* Test the pagination

* Test centering around a point

* Handle `0` sizes

Co-Authored-By: Zacqary Adam Xeper <Zacqary@users.noreply.github.com>

* Add highlights endpoint

* Refactor `processCursor`

* Tweak cursor handling in the routes

* Refine `LogEntry` type

* Add tests for highlights endpoint

* Tweak the types for the LogEntry

Co-authored-by: Zacqary Adam Xeper <Zacqary@users.noreply.github.com>
@afgomez afgomez added the backport This PR is a backport of another PR label Jan 6, 2020
@afgomez afgomez mentioned this pull request Jan 7, 2020
@afgomez afgomez force-pushed the backport/7.x/pr-53798 branch from c62cdf5 to 28752fb Compare January 7, 2020 11:13
ES behaves differently in master and in 7.x, causing the test to fail in
the latter.
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

History

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

@afgomez afgomez merged commit 8bb872f into elastic:7.x Jan 7, 2020
@afgomez afgomez deleted the backport/7.x/pr-53798 branch January 7, 2020 14:47
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