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

Miscellaneous App / Arch items 👉 New Platform #51560

Closed
40 tasks done
lukeelmers opened this issue Nov 25, 2019 · 3 comments
Closed
40 tasks done

Miscellaneous App / Arch items 👉 New Platform #51560

lukeelmers opened this issue Nov 25, 2019 · 3 comments

Comments

@lukeelmers
Copy link
Member

lukeelmers commented Nov 25, 2019

Part of #31968

So far, the bulk of the code we have focused on migrating to the new platform either lives as a plugin or as a ui/public module. However, there are a handful of pieces such as utilities and one-off server endpoints which will also need to be migrated.

The purpose of this meta issue is to track these miscellaneous, "smaller" items that @elastic/kibana-app-arch will need to migrate to the new platform. This ensures we have a comprehensive list of everything we are aware of that needs to be reviewed.

src/legacy/ui/public

src/legacy/server

src/legacy/core_plugins

x-pack/legacy/plugins [3]

Other Tasks

Blocked items
These are items which are currently blocked by other work and may need to happen later in 7.x

@jasonrhodes
Copy link
Member

As far as I can tell, QueryString.encode is identical to querystring.stringify and QueryString.decode the same as querystring.parse, using the querystring lib that we already import in Kibana (we were using it in APM, at least). I've switched the infra (metrics/logs) usages of these functions over to querystring and haven't had problems yet. If others need to be unblocked on that util import, that's one option.

@flash1293
Copy link
Contributor

@lukeelmers Not sure whether this is the right place, but src/legacy/ui/public/visualize/loader/pipeline_helpers/utilities.ts is currently referenced by Lens. We should think about how to expose it in the future.

@lukeelmers
Copy link
Member Author

@jasonrhodes Thanks for the tip on querystring. We will try updating uses of the legacy QueryString across Kibana and see how it goes. Tracking that in #56704

@flash1293 I think @ppisljar had planned on moving that into the visualizations or expressions plugins, but he'd be able to confirm.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants