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

[Maps] Get basic structure of NP client shim in place #52551

Merged
merged 8 commits into from
Dec 12, 2019

Conversation

kindsun
Copy link
Contributor

@kindsun kindsun commented Dec 9, 2019

Puts the basic structure for the Maps client NP shim in place. I determined this to be a dependency (but it also needed to happen anyway) when I was adding in capability to check & use a license from the client to send to the EMS server.

For the most part, this preserves the structure of the original Maps App client setup, with a couple changes:

  • Pull routes init into a separate file
  • Add legacy.ts shim
  • Pass uiModules as a legacy service for use in setup
  • General NP plugin structure changes

In successive PRs, uiModules and other legacy services will need to be either replaced or handled more extensively in the shim

@kindsun kindsun added Feature:New Platform [Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation v8.0.0 v7.6.0 labels Dec 9, 2019
@elasticmachine
Copy link
Contributor

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

@kindsun kindsun marked this pull request as ready for review December 10, 2019 15:18
@kindsun kindsun added the release_note:skip Skip the PR/issue when compiling release notes label Dec 10, 2019
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, tested sample datasets

@kindsun
Copy link
Contributor Author

kindsun commented Dec 12, 2019

@elasticmachine merge upstream

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

History

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

@kindsun kindsun merged commit 3d10370 into elastic:master Dec 12, 2019
kindsun pushed a commit to kindsun/kibana that referenced this pull request Dec 12, 2019
* Move ui routes init to separate file

* Add index and general maps plugin template

* Add legacy shim. Pass uimodules in as legacy service

* Move ts-ignore

* Fix N/A ts error on uiModules

* Point index file toward legacy file. Minor NP additions
kindsun pushed a commit that referenced this pull request Dec 12, 2019
* Move ui routes init to separate file

* Add index and general maps plugin template

* Add legacy shim. Pass uimodules in as legacy service

* Move ts-ignore

* Fix N/A ts error on uiModules

* Point index file toward legacy file. Minor NP additions
timductive pushed a commit to timductive/kibana that referenced this pull request Dec 16, 2019
* Move ui routes init to separate file

* Add index and general maps plugin template

* Add legacy shim. Pass uimodules in as legacy service

* Move ts-ignore

* Fix N/A ts error on uiModules

* Point index file toward legacy file. Minor NP additions
jloleysens added a commit to jloleysens/kibana that referenced this pull request Dec 30, 2019
…aved-objects

* upstream/master: (134 commits)
  [Dashboard] Add visualization from dasbhoard empty screen (elastic#52670)
  Print out agent debugging links during CI (elastic#52812)
  Add babel-plugin-styled-components to webpack config (elastic#52862)
  [Console] Fix load from remote (elastic#52814)
  Ensure APM agent config file path respects CWD (elastic#52880)
  [Watcher] Removed overwritten property (elastic#49998)
  [Data Plugin]: Remove `export *` for common code from public/server index files (elastic#52821)
  Hide stderr git output during APM agent configuration (elastic#52878)
  Polish migration.md (elastic#52764)
  Change ajax_stream to use new-line delimited JSON (elastic#52797)
  Stabilize dashboard save modal functional test (elastic#52761)
  [Discover] Place tooltip at bottom of filter button (elastic#52720)
  Disable/enable filter with click+shift on a filter badge (elastic#52751)
  [APM] Make client-side routes static (elastic#52574)
  [Maps] Get basic structure of NP client shim in place (elastic#52551)
  update chromedriver to 79 (elastic#52784)
  [DOCS] Adds example of assigning roles in Reporting (elastic#52757)
  Add instructions for setting up remote clusters needed for CCS and CCR (elastic#52796)
  [docs] max-old-space-size (elastic#52310)
  [Monitoring] Fix 7.5 cloud test issues (elastic#51781)
  ...
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:New Platform release_note:skip Skip the PR/issue when compiling release notes v7.6.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants