Skip to content

Management: Index Pattern directory rename / correction#26858

Merged
mattkime merged 4 commits intoelastic:masterfrom
mattkime:mgmt_rename_index_patterns_dir
Dec 13, 2018
Merged

Management: Index Pattern directory rename / correction#26858
mattkime merged 4 commits intoelastic:masterfrom
mattkime:mgmt_rename_index_patterns_dir

Conversation

@mattkime
Copy link
Contributor

@mattkime mattkime commented Dec 9, 2018

Summary

Pretty much one git mv command - Index Pattern code should be in a dir named index_patterns instead of indices

@mattkime mattkime requested a review from jen-huang December 9, 2018 04:37
@elasticmachine
Copy link
Contributor

💔 Build Failed

@mattkime
Copy link
Contributor Author

mattkime commented Dec 9, 2018

retest

@elasticmachine
Copy link
Contributor

💔 Build Failed

@mattkime
Copy link
Contributor Author

mattkime commented Dec 9, 2018

retest

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Contributor

@jen-huang jen-huang left a comment

Choose a reason for hiding this comment

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

LGTM! Smoke tested locally.

@mattkime mattkime merged commit e316963 into elastic:master Dec 13, 2018
mattkime added a commit to mattkime/kibana that referenced this pull request Dec 13, 2018
mattkime added a commit that referenced this pull request Dec 13, 2018
)

* correct directory name

* fix scss reference
@mattkime mattkime added v7.0.0 v6.7.0 Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// labels Feb 4, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// v6.7.0 v7.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants