-
Notifications
You must be signed in to change notification settings - Fork 3.5k
Doc: Provide accurate post-geoip lookup example #18092
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
🤖 GitHub commentsExpand to view the GitHub comments
Just comment with:
|
This pull request does not have a backport label. Could you fix it @karenzone? 🙏
|
🔍 Preview links for changed docs |
@kaisecheng, a user pointed out that the "after" example is the same as the "before" example. Will you help me come up with a proper example, please? |
@karenzone Thx. I updated the example and push to the branch. |
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.
Renders as expected, and looks great!
Thank you, @kaisecheng.
Co-authored-by: Kaise Cheng <[email protected]> (cherry picked from commit 23d06e6)
Co-authored-by: Kaise Cheng <[email protected]> (cherry picked from commit 23d06e6)
(cherry picked from commit 23d06e6) Co-authored-by: Karen Metts <[email protected]> Co-authored-by: Kaise Cheng <[email protected]>
(cherry picked from commit 23d06e6) Co-authored-by: Karen Metts <[email protected]> Co-authored-by: Kaise Cheng <[email protected]>
Closes: elastic/docs-content#2782