Skip to content

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

Merged
kindsun merged 8 commits intoelastic:masterfrom
kindsun:np-inital-client-shim
Dec 12, 2019
Merged

[Maps] Get basic structure of NP client shim in place#52551
kindsun merged 8 commits intoelastic:masterfrom
kindsun:np-inital-client-shim

Conversation

@kindsun
Copy link
Copy Markdown
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 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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor Author

kindsun commented Dec 12, 2019

@elasticmachine merge upstream

@elasticmachine
Copy link
Copy Markdown
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
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

Feature:New Platform release_note:skip Skip the PR/issue when compiling release notes Team:Geo Former Team Label for Geo Team. Now use Team:Presentation v7.6.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants