Skip to content

[Watcher] Reverting incorrectly removed scope parameters#37875

Merged
jgowdyelastic merged 2 commits into
elastic:masterfrom
jgowdyelastic:watcher-fixing-incorrectly-removed-scope-parameters
Jun 4, 2019
Merged

[Watcher] Reverting incorrectly removed scope parameters#37875
jgowdyelastic merged 2 commits into
elastic:masterfrom
jgowdyelastic:watcher-fixing-incorrectly-removed-scope-parameters

Conversation

@jgowdyelastic
Copy link
Copy Markdown
Member

@jgowdyelastic jgowdyelastic commented Jun 3, 2019

PR #37051 removed scope parameters in angular directives which are necessary for the directive to work.

Relates to #37873

Updated to match changes in #37922

@jgowdyelastic jgowdyelastic added bug Fixes for quality problems that affect the customer experience backport pending Feature:Watcher v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.3.0 labels Jun 3, 2019
@jgowdyelastic jgowdyelastic self-assigned this Jun 3, 2019
Copy link
Copy Markdown
Contributor

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

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

LGTM

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@jgowdyelastic
Copy link
Copy Markdown
Member Author

retest

@jgowdyelastic jgowdyelastic force-pushed the watcher-fixing-incorrectly-removed-scope-parameters branch from 81adebb to 1592a65 Compare June 3, 2019 15:15
@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@jgowdyelastic
Copy link
Copy Markdown
Member Author

retest

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@jgowdyelastic
Copy link
Copy Markdown
Member Author

retest

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@jgowdyelastic jgowdyelastic merged commit dce42af into elastic:master Jun 4, 2019
@jgowdyelastic jgowdyelastic deleted the watcher-fixing-incorrectly-removed-scope-parameters branch June 4, 2019 09:00
jgowdyelastic added a commit that referenced this pull request Jun 4, 2019
* [Watcher] Reverting incorrectly removed scope parameters

* removing  no-unused-vars comments
jgowdyelastic added a commit to jgowdyelastic/kibana that referenced this pull request Jun 27, 2019
* [Watcher] Reverting incorrectly removed scope parameters

* removing  no-unused-vars comments
jgowdyelastic added a commit that referenced this pull request Jun 28, 2019
…9778)

* [Watcher] Reverting incorrectly removed scope parameters

* removing  no-unused-vars comments
patrykkopycinski pushed a commit to patrykkopycinski/kibana that referenced this pull request May 6, 2026
* [Watcher] Reverting incorrectly removed scope parameters

* removing  no-unused-vars comments
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:Watcher release_note:skip Skip the PR/issue when compiling release notes v7.3.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants