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

[Painless Lab] Moving out of Beta #61809

Open
11 tasks
jloleysens opened this issue Mar 30, 2020 · 2 comments
Open
11 tasks

[Painless Lab] Moving out of Beta #61809

jloleysens opened this issue Mar 30, 2020 · 2 comments
Labels
enhancement New value added to drive a business result Feature:Dev Tools Feature:Painless Lab Dev tool for learning Painless Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more

Comments

@jloleysens
Copy link
Contributor

jloleysens commented Mar 30, 2020

Summary

With #57538 merged, this meta issue will be used for tracking any additional feedback or work needed for moving Painless Lab out of beta.

Current items to address

After we have worked through these items we should be able to remove the beta. They are listed in order of importance (determined by the amount of work and the potential value it will contribute).

Check off each item as we address them and add a link to any PR related to addressing the work.

Features

  • Prepopulate index-selection field with a list of available indices. We also want to create a flyout that lets you preview the mappings of each index to help you select the one you want.
  • The parameters field should be a component that lets you add key-value pairs via a list editor. There can be a button that lets you switch back and forth between the KV list mode and JSON editing mode.
  • Syntax checking and highlighting for JSON text editors and implement regex syntax highlighting in Painless editor (per Create Painless Lab app #57538 (comment))
  • Allow the user to opt out of live updating and use a "Submit" button instead, per Refine Painless Playground UI/UX kertal/kibana#5 (comment)

Fixes

Tests and docs

  • Add a page to the Elastic docs
  • Add API integration tests for the execute endpoint. Check that, for instance, invalid Painless still returns 200 with error details as opposed to 400.
  • Add Jest component integration tests (or at least a smoke test for the main container)
  • Create a test rail plan

Other

One interesting suggestion mentioned in the original painless lab pr: #57538 (comment). Provide user feedback inline rather than in a separate, always on, output area.

@jloleysens jloleysens added Feature:Dev Tools Meta Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more Feature:Painless Lab Dev tool for learning Painless labels Mar 30, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/es-ui (Team:Elasticsearch UI)

@alisonelizabeth alisonelizabeth removed the Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more label Sep 16, 2024
@botelastic botelastic bot added the needs-team Issues missing a team label label Sep 16, 2024
@alisonelizabeth alisonelizabeth added Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more and removed Feature:Dev Tools Meta Feature:Painless Lab Dev tool for learning Painless needs-team Issues missing a team label labels Sep 16, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-management (Team:Kibana Management)

@alisonelizabeth alisonelizabeth added Feature:Painless Lab Dev tool for learning Painless enhancement New value added to drive a business result Feature:Dev Tools labels Sep 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New value added to drive a business result Feature:Dev Tools Feature:Painless Lab Dev tool for learning Painless Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more
Projects
None yet
Development

No branches or pull requests

3 participants