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

Fix "my second guide" title, and cross-link #35571

Merged
merged 1 commit into from
Sep 20, 2023

Conversation

holly-cummins
Copy link
Contributor

I don't know how we didn't notice this in preview, but despite having a title, the title for "my second application" is generated nonsense:

image

The adoc looks fine in an adoc editor, so I'm not sure what's wrong. I've tried removing the id to see if that helps.

I've also cross-linked from the first application guide to the second application guide.

@gsmet
Copy link
Member

gsmet commented Aug 25, 2023

Let me do some experiments on the website before merging this.

@github-actions
Copy link

github-actions bot commented Aug 25, 2023

🎊 PR Preview e929e65 has been successfully built and deployed to https://quarkus-pr-main-35571-preview.surge.sh/version/main/guides/

@@ -4,17 +4,20 @@ and pull requests should be submitted there:
https://github.com/quarkusio/quarkus/tree/main/docs/src/main/asciidoc
////

[id="getting-started-dev-services-tutorial"]
Copy link
Member

Choose a reason for hiding this comment

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

You can keep that, what you need is to get rid of the new line above.

See quarkusio/quarkusio.github.io@a30a2d5 and https://quarkus.io/guides/getting-started-dev-services

…d try and sort out title of second application guide.
@gsmet gsmet force-pushed the second-application-guide-tweaks branch from 18f78e4 to 966c4df Compare September 20, 2023 07:39
Copy link
Member

@gsmet gsmet left a comment

Choose a reason for hiding this comment

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

I fixed the issue, it should be good to go now.

@gsmet gsmet added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Sep 20, 2023
@holly-cummins
Copy link
Contributor Author

Thank you, @gsmet!

@gsmet gsmet merged commit c2f9d46 into quarkusio:main Sep 20, 2023
5 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.5 - main milestone Sep 20, 2023
@quarkus-bot quarkus-bot bot removed the triage/waiting-for-ci Ready to merge when CI successfully finishes label Sep 20, 2023
@gsmet gsmet added triage/waiting-for-ci Ready to merge when CI successfully finishes triage/backport? labels Sep 20, 2023
@gsmet gsmet modified the milestones: 3.5 - main, 3.4.2 Oct 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation triage/waiting-for-ci Ready to merge when CI successfully finishes
Projects
Development

Successfully merging this pull request may close these issues.

2 participants