Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New platform move doc table and doc viewer to kibana\discover #34231

Merged

Conversation

lizozom
Copy link
Contributor

@lizozom lizozom commented Mar 31, 2019

Summary

Moved doc table and doc viewer from ui/public to kibana plugin

Checklist

Use strikethroughs to remove checklist items you don't feel are applicable to this PR.

- [ ] This was checked for cross-browser compatibility, including a check against IE11
- [ ] Any text added follows EUI's writing guidelines, uses sentence case text and includes [i18n support](https:/elastic/kibana/blob/master/packages/kbn-i18n/README.md)
- [ ] Documentation was added for features that require explanation or tutorials
- [ ] Unit or functional tests were updated or added to match the most common scenarios
- [ ] This was checked for keyboard-only and screenreader accessibility

@lizozom lizozom added the chore label Mar 31, 2019
@lizozom lizozom requested a review from a team as a code owner March 31, 2019 12:04
@stacey-gammon stacey-gammon mentioned this pull request Mar 31, 2019
94 tasks
@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@lizozom lizozom added Feature:New Platform Team:Visualizations Visualization editors, elastic-charts and infrastructure v8.0.0 v7.2.0 labels Mar 31, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app

@lizozom lizozom self-assigned this Mar 31, 2019
Liza Katz added 3 commits April 2, 2019 13:34
- saved object finder
- paginated selectable list
from partiasls to directives/partials
@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@lizozom
Copy link
Contributor Author

lizozom commented Apr 3, 2019

Discover technically supports showing paginated results (instead of an infinite scroll)
However, I can't find anyone setting the infiniteScroll to any value, but the default.
Do you know if this behavior is still supported or is it a deprecated remainder, that can be removed?

@stacey-gammon (your name is on that line) @AlonaNadler

https:/elastic/kibana/search?q=infiniteScroll&unscoped_q=infiniteScroll

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Member

@lukeelmers lukeelmers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Just had one question

@elasticmachine
Copy link
Contributor

💔 Build Failed

@lizozom
Copy link
Contributor Author

lizozom commented Apr 7, 2019

retest

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@lizozom lizozom merged commit 4dd7f17 into elastic:master Apr 7, 2019
lizozom added a commit to lizozom/kibana that referenced this pull request Apr 7, 2019
…c#34231)

* Moved doc table to kibana/discover

* moved doc viewer to Kibana app (used by docs and discover)

* Moved
- saved object finder
- paginated selectable list
from partiasls to directives/partials

* Moved paginate controls directive to directives/partials
lizozom added a commit that referenced this pull request Apr 7, 2019
#34678)

* Moved doc table to kibana/discover

* moved doc viewer to Kibana app (used by docs and discover)

* Moved
- saved object finder
- paginated selectable list
from partiasls to directives/partials

* Moved paginate controls directive to directives/partials
chandlerprall pushed a commit to chandlerprall/kibana that referenced this pull request Apr 15, 2019
…c#34231)

* Moved doc table to kibana/discover

* moved doc viewer to Kibana app (used by docs and discover)

* Moved
- saved object finder
- paginated selectable list
from partiasls to directives/partials

* Moved paginate controls directive to directives/partials
@lizozom lizozom deleted the new-platform-move-doc-table-to-discover branch April 21, 2019 10:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Feature:New Platform Team:Visualizations Visualization editors, elastic-charts and infrastructure v7.2.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants