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

Exposed AddMessageVariables as separate component #63007

Conversation

YulNaumenko
Copy link
Contributor

Exposed AddMessageVariables as separate component and added styles to allow to handle bigger list of messageVariables

Resolve #62909

… allow to handle bigger list of messageVariables
@YulNaumenko YulNaumenko added bug Fixes for quality problems that affect the customer experience Feature:Alerting v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.7.0 Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v7.8.0 labels Apr 8, 2020
@YulNaumenko YulNaumenko requested review from a team as code owners April 8, 2020 19:03
@YulNaumenko YulNaumenko self-assigned this Apr 8, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-alerting-services (Team:Alerting Services)

Copy link
Contributor

@snide snide left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small quick change.

@YulNaumenko YulNaumenko requested a review from snide April 8, 2020 20:47
Copy link
Contributor

@snide snide left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

History

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

@mikecote mikecote self-requested a review April 9, 2020 14:34
Copy link
Contributor

@mikecote mikecote left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@YulNaumenko YulNaumenko merged commit 80384c3 into elastic:master Apr 9, 2020
YulNaumenko added a commit to YulNaumenko/kibana that referenced this pull request Apr 9, 2020
* Exposed AddMessageVariables as separate component and added styles to allow to handle bigger list of messageVariables

* Fixed failing tests and styles

* Fixed due to comments
YulNaumenko added a commit to YulNaumenko/kibana that referenced this pull request Apr 9, 2020
* Exposed AddMessageVariables as separate component and added styles to allow to handle bigger list of messageVariables

* Fixed failing tests and styles

* Fixed due to comments
YulNaumenko added a commit that referenced this pull request Apr 9, 2020
* Exposed AddMessageVariables as separate component and added styles to allow to handle bigger list of messageVariables

* Fixed failing tests and styles

* Fixed due to comments
jloleysens added a commit to jloleysens/kibana that referenced this pull request Apr 9, 2020
…chore/put-all-xjson-together

* 'master' of github.com:elastic/kibana:
  [EPM] Update UI copy to use `integration` (elastic#63077)
  [NP] Inline buildPointSeriesData and buildHierarchicalData dependencies (elastic#61575)
  [Maps] create NOT EXISTS filter for tooltip property with no value (elastic#62849)
  [Endpoint] Add link to Logs UI to the Host Details view (elastic#62852)
  [UI COPY] Fixes typo in max_shingle_size for search_as_you_type (elastic#63071)
  [APM] docs: add alerting examples for APM (elastic#62864)
  [EPM] Change PACKAGES_SAVED_OBJECT_TYPE id (elastic#62818)
  docs: fix rendering of bulleted list (elastic#62855)
  Exposed AddMessageVariables as separate component (elastic#63007)
  Add Data - Adding cloud reset password link to cloud instructions (elastic#62835)
  [ML] DF Analytics:  update memory estimate after adding exclude fields (elastic#62850)
  [Table Vis] Fix visualization overflow (elastic#62630)
  [Endpoint][EPM] Endpoint depending on ingest manager to initialize (elastic#62871)
  [Remote clusters] Fix flaky jest tests (elastic#58768)
  [Discover] Hide time picker when an indexpattern without timefield is selected (elastic#62134)
  Move search source parsing and serializing to data (elastic#59919)
  [ML] Functional tests - stabilize typing in mml input (elastic#63091)
  [data.search.aggs]: Clean up TimeBuckets implementation (elastic#62123)
  [ML] Functional transform tests - stabilize source selection (elastic#63087)
  add embed flag to saved object url as well (elastic#62926)

# Conflicts:
#	x-pack/plugins/triggers_actions_ui/public/application/components/builtin_action_types/es_index.tsx
YulNaumenko added a commit that referenced this pull request Apr 10, 2020
)

* Exposed AddMessageVariables as separate component (#63007)

* Exposed AddMessageVariables as separate component and added styles to allow to handle bigger list of messageVariables

* Fixed failing tests and styles

* Fixed due to comments

* Fixed functional test assert due to the changes

* fixed failing func tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience Feature:Alerting release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v7.7.0 v7.8.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Alerting] Allow ActionForm to handle bigger list of messageVariables
5 participants