-
Notifications
You must be signed in to change notification settings - Fork 30
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
[Manage data] Refine Mapping section #408
Conversation
@@ -8,7 +8,7 @@ mapped_pages: | |||
One of the most important features of {{es}} is that it tries to get out of your way and let you start exploring your data as quickly as possible. To index a document, you don’t have to first create an index, define a mapping type, and define your fields — you can just index a document and the index, type, and fields will display automatically: | |||
|
|||
```console | |||
PUT data/_doc/1 <1> | |||
PUT data/_doc/1 <1> |
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 noticed that the "1" doesn't render properly in this code block and in the bullet after. It seems to be formatted the same as other pages where it's rendering like it's supposed to. I tried to troubleshoot this but I'm at a loss. Would appreciate any ideas
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.
@wajihaparvez Good question! If you change "console" to "json" I think it'll work (I tried it locally and that did the trick). That is, it seems to prevent the Markdown build from interpreting the <1>
as part of the code example.
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.
Thank you @kilfoyle, that worked!
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.
LGTM! 🦕
Very nice!
Closes #370
Preview here
Summary
This PR includes the following updates to the Mapping section:
Note to reviewer: Please see my comment about a wonky formatting error in one of the lift-and-shift pages. Insight is appreciated 🙏