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

[mdx → asciidoc] Add AsciiDoc serverless files #168

Merged
merged 26 commits into from
Nov 5, 2024

Conversation

colleenmcginnis
Copy link
Contributor

@colleenmcginnis colleenmcginnis commented Nov 4, 2024

Related to elastic/docs#3096

I did a full review where I looked at:

  • The diff between the MDX and AsciiDoc file for each page and confirmed that the only changes were to the syntax.
  • The rendered result of the both the MDX version and AsciiDoc version of each page side-by-side and confirmed that they result in the same structure.

This PR adds AsciiDoc files and images to replace DocIcon. A few MDX files are edited where the MDX syntax was incorrect. MDX files will be deleted in a follow-up PR.

You can see the result in this preview: https://docs_bk_3096.docs-preview.app.elstc.co/guide/en/serverless/current/intro.html

@colleenmcginnis colleenmcginnis self-assigned this Nov 4, 2024
@elasticdocs
Copy link
Collaborator

elasticdocs commented Nov 4, 2024

🚀 Built elastic-dot-co-docs-preview-docs successfully!

Issues? Visit #next-docs in Slack

@bmorelli25
Copy link
Member

run docs-build

@colleenmcginnis colleenmcginnis marked this pull request as ready for review November 5, 2024 16:03
@colleenmcginnis colleenmcginnis requested a review from a team as a code owner November 5, 2024 16:03
Copy link
Contributor

@georgewallace georgewallace left a comment

Choose a reason for hiding this comment

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

reviewed preview and every page renders.

@colleenmcginnis colleenmcginnis merged commit 80ba445 into main Nov 5, 2024
2 checks passed
@bmorelli25 bmorelli25 deleted the mdx-to-asciidoc branch November 5, 2024 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants