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

Website 2.0 #1207

Closed
wants to merge 147 commits into from
Closed

Website 2.0 #1207

wants to merge 147 commits into from

Conversation

insectengine
Copy link
Collaborator

** If you are updating a guide, please submit your pull request to the main repository: https://github.com/quarkusio/quarkus/tree/main/docs/src/main/asciidoc **

Comment on lines +13 to +15
<pre class="highlight"><code>
curl -Ls https://sh.jbang.dev | bash -s - app install --fresh --force quarkus@quarkusio
</code></pre>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<pre class="highlight"><code>
curl -Ls https://sh.jbang.dev | bash -s - app install --fresh --force quarkus@quarkusio
</code></pre>
<pre class="highlight"><code>curl -Ls https://sh.jbang.dev | bash -s - app install --fresh --force quarkus@quarkusio</code></pre>

all the code sections should not have start/end empty lines as that result in "copy" button giving you 3 lines instead of 1 which most terminals complains about.

@@ -0,0 +1,8 @@
---
Copy link
Member

Choose a reason for hiding this comment

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

do we wanna continue doing this event approach or instead just use the Event category on github discussions? or some mix?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I was thinking a mix since we already have the world tour on the site. We could use the site to list major events we are attending/sponsoring but use the GH Discussion section for additional communication.

I've removed the extensions section from the navigation and the corresponding files from the build so they won't be generated. I also added a section to the top of the new getting started page that defines the Quarkus pre-requests.
Updated Events section content., Updated training to include link to discussions, updated getting started to include pre-requests.
ia3andy and others added 27 commits November 23, 2021 11:56
Apply suggestions from code review

Co-authored-by: Guillaume Smet <[email protected]>
…or the JWT guide from the web category

Signed-off-by: Nathan Erwin <[email protected]>
I've removed the extensions section from the navigation and the corresponding files from the build so they won't be generated. I also added a section to the top of the new getting started page that defines the Quarkus pre-requests.
Updated Events section content., Updated training to include link to discussions, updated getting started to include pre-requests.
@github-actions
Copy link

😭 Deploy PR Preview e87f744 failed. Build logs

🤖 By surge-preview

@insectengine insectengine deleted the Website-2.0 branch November 23, 2021 19:39
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.