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

Add Data Paste Sample Logs Screen #6005

Merged
merged 8 commits into from
Feb 4, 2016
Merged

Conversation

Bargs
Copy link
Contributor

@Bargs Bargs commented Jan 26, 2016

Requires #5790

WIP PR to implement the sample log screen in #5974

@Bargs Bargs self-assigned this Jan 26, 2016
@Bargs Bargs added the review label Feb 2, 2016
@Bargs Bargs assigned spalger and unassigned Bargs Feb 2, 2016
@@ -1,12 +1,36 @@
var modules = require('ui/modules');
var template = require('plugins/kibana/settings/sections/indices/add_data_steps/paste_samples_step.html');
const modules = require('ui/modules');
Copy link
Contributor

Choose a reason for hiding this comment

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

Mind using import here? several prs are working on moving everything this way.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure thing. I'll make sure to use import in the future as well.

@spalger
Copy link
Contributor

spalger commented Feb 4, 2016

LGTM

@spalger spalger assigned Bargs and unassigned spalger Feb 4, 2016
Bargs pushed a commit that referenced this pull request Feb 4, 2016
Add Data Paste Sample Logs Screen
@Bargs Bargs merged commit fd1d480 into elastic:feature/ingest Feb 4, 2016
@Bargs Bargs mentioned this pull request Feb 12, 2016
15 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants