-
Notifications
You must be signed in to change notification settings - Fork 40
Add a link from winston ECS logging to a Cloud tutorial doc #100
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
Conversation
💚 Build Succeeded
Expand to view the summary
Build stats
Trends 🧪 |
docs/winston.asciidoc
Outdated
@@ -265,3 +265,5 @@ const logger = winston.createLogger({ | |||
// ... | |||
}) | |||
---- | |||
|
|||
To learn more, try out our tutorial using Node.js ECS logging with winston: {cloud}/ec-getting-started-search-use-cases-node-logs.html[Ingest logs from a Node.js web application using Filebeat]. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this would be better in the "Setup" section up higher. Could you move this up to just above these lines:
[float]
[[winston-usage]]
=== Usage
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good idea. I moved the link up to the "Configure Filebeat" section.
Move the tutorial link up to the `Configure Filebeat` section.
jenkins run the tests |
@elasticmachine run elasticsearch-ci/docs |
Thanks for approving this! I'm not able to merge it but if there's anything I can do to help, let me know. |
Ah, I see it's merged now. Thanks @trentm! |
Rel: #99
Adds a link at the bottom of this Node.js winston ECS logging page to a similar tutorial in the Cloud docs.