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

[File upload] Move File Upload to New Platform #58550

Merged
merged 11 commits into from
Mar 4, 2020

Conversation

kindsun
Copy link
Contributor

@kindsun kindsun commented Feb 25, 2020

Final move of File Upload to New Platform

Also updated:

  • Telemetry mappings registration
  • Maps usage of File Upload. Now pulled in through x-pack/legacy/plugins/maps/public/plugin.ts like other plugins

@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-gis (Team:Geo)

@kindsun kindsun marked this pull request as ready for review February 26, 2020 22:18
@kindsun kindsun requested a review from a team as a code owner February 26, 2020 22:18
@kindsun kindsun added the release_note:skip Skip the PR/issue when compiling release notes label Mar 2, 2020
Copy link
Contributor

@thomasneirynck thomasneirynck left a comment

Choose a reason for hiding this comment

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

code review, and tested in Chrome. Some naming nits.

Very cool!!!

x-pack/legacy/plugins/maps/public/kibana_services.js Outdated Show resolved Hide resolved
x-pack/legacy/plugins/maps/public/kibana_services.js Outdated Show resolved Hide resolved
* Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
* or more contributor license agreements. Licensed under the Elastic License;
* you may not use this file except in compliance with the Elastic License.
*/
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not really sure how to test ..

Copy link
Contributor Author

Choose a reason for hiding this comment

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

There's still a global telemetry test covering it here that should fail if telemetry isn't being generated. Checking locally produces this result as expected:

image

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

History

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

@kindsun kindsun merged commit c4458ca into elastic:master Mar 4, 2020
kindsun pushed a commit to kindsun/kibana that referenced this pull request Mar 4, 2020
* Move file upload to np. Some additional mods & removals

* Consume file upload from NP in maps and pass to kibana services

* Register telemetry mappings

* Init indexPatternService in start method

* Fix type check issues. Add missing prop to telemetry

* Update i18n path

* Review feedback
kindsun pushed a commit that referenced this pull request Mar 4, 2020
* Move file upload to np. Some additional mods & removals

* Consume file upload from NP in maps and pass to kibana services

* Register telemetry mappings

* Init indexPatternService in start method

* Fix type check issues. Add missing prop to telemetry

* Update i18n path

* Review feedback
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation Feature:File Upload Feature:NP Migration release_note:skip Skip the PR/issue when compiling release notes v7.7.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants