-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
Support "Deprecated" label in advanced settings #54539
Support "Deprecated" label in advanced settings #54539
Conversation
mark courier:batchSearches as deprecated
Pinging @elastic/kibana-app-arch (Team:AppArch) |
@elasticmachine merge upstream |
d719885
to
421981d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
code lgtm
src/legacy/core_plugins/kibana/public/management/sections/settings/components/field/field.js
Outdated
Show resolved
Hide resolved
@stacey-gammon @lukasolson What do you think? |
…/deprecation-label
This is great! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The UI LGTM. I saw just a couple things in the code and as mentioned, this still needs a deprecation statement in the docs.
src/legacy/core_plugins/kibana/public/management/sections/settings/components/field/field.js
Outdated
Show resolved
Hide resolved
Translate click aria label Use docLinks service
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 on the UI. Will you be adding the deprecation warning in the docs in this PR?
…/deprecation-label
This reverts commit c9512ce.
…/deprecation-label
💚 Build SucceededHistory
To update your PR or re-run it, just comment with: |
* Support deprecating label in advanced settings mark courier:batchSearches as deprecated * jest update * Add deprecation to UiSettingsParams type Translate click aria label Use docLinks service * Rename doc link * Remove url option from DeprecationSettings * Simplify code * Updated docs * Revert "Updated docs" This reverts commit c9512ce. * snapshots * docs
* Support deprecating label in advanced settings mark courier:batchSearches as deprecated * jest update * Add deprecation to UiSettingsParams type Translate click aria label Use docLinks service * Rename doc link * Remove url option from DeprecationSettings * Simplify code * Updated docs * Revert "Updated docs" This reverts commit c9512ce. * snapshots * docs
// Warning: (ae-forgotten-export) The symbol "DeprecationSettings" needs to be exported by the entry point index.d.ts | ||
deprecation?: DeprecationSettings; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor, but DeprecationSettings
should be exported
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* upstream/master: (26 commits) Take page offset into account too (elastic#54567) [APM] Support error.{log,exception}.stacktrace.classname (elastic#54577) Np migration tsvb route validation (elastic#51850) [ML] MML calculator enhancements for multi-metric job wizard (elastic#54573) [SIEM] Fix Inspect query 'request timestamp' value changes when curso… (elastic#54223) Fix chromeless NP apps not using full page width (elastic#54550) Remove extraneous public import to prevent failing Kibana startup (elastic#54676) [Uptime] Temporarily skip flakey tests (elastic#54675) Skip failing uptime tests Create UI for alerting and actions plugin (elastic#48959) [dev/build/sass] build stylesheets for disabled plugins too (elastic#54654) [SIEM] Use bulk actions API when updating or deleting rules (elastic#54521) Support "Deprecated" label in advanced settings (elastic#54539) [Maps] add text halo color and width style properties (elastic#53827) Service Map Data API at Runtime (elastic#54027) [SIEM] Detection Engine Create Rule Design Review #1 (elastic#54442) Skip flaky test [Canvas] Enable Embeddable maps (elastic#53971) [SIEM][Detection Engine] Increases the number or rules you can view on a single page (elastic#54628) uiSettings - use validation field for image field maxSize (elastic#54522) ...
* Support deprecating label in advanced settings mark courier:batchSearches as deprecated * jest update * Add deprecation to UiSettingsParams type Translate click aria label Use docLinks service * Rename doc link * Remove url option from DeprecationSettings * Simplify code * Updated docs * Revert "Updated docs" This reverts commit c9512ce. * snapshots * docs
Summary
courier:batchSearches
as deprecatedChecklist
Use
strikethroughsto remove checklist items you don't feel are applicable to this PR.Unit or functional tests were updated or added to match the most common scenariosFor maintainers
This was checked for breaking API changes and was labeled appropriatelyThis includes a feature addition or change that requires a release note and was labeled appropriately