Skip to content

[ML] Reverting incorrectly removed scope parameters#37873

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

[ML] Reverting incorrectly removed scope parameters#37873
jgowdyelastic merged 2 commits into
elastic:masterfrom
jgowdyelastic:removing-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 #37875

Updated to match changes in #37922

@jgowdyelastic jgowdyelastic added bug Fixes for quality problems that affect the customer experience review backport pending :ml v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.3.0 labels Jun 3, 2019
@jgowdyelastic jgowdyelastic requested a review from a team as a code owner June 3, 2019 13:05
@jgowdyelastic jgowdyelastic self-assigned this Jun 3, 2019
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/ml-ui

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

Copy link
Copy Markdown
Contributor

@alvarezmelissa87 alvarezmelissa87 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 removing-incorrectly-removed-scope-parameters branch from fe713e9 to 399c1fc Compare June 3, 2019 15:15
@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@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 5016a63 into elastic:master Jun 4, 2019
@jgowdyelastic jgowdyelastic deleted the removing-incorrectly-removed-scope-parameters branch June 4, 2019 09:00
jgowdyelastic added a commit that referenced this pull request Jun 4, 2019
* [ML] 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
* [ML] Reverting incorrectly removed scope parameters

* removing  no-unused-vars comments
jgowdyelastic added a commit that referenced this pull request Jun 27, 2019
* [ML] 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
* [ML] 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 :ml release_note:skip Skip the PR/issue when compiling release notes review v7.3.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants