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

Remove link to Dev UI v1 as it is now gone #36116

Merged

Conversation

Sap004
Copy link
Contributor

@Sap004 Sap004 commented Sep 23, 2023

fixes #36109

@quarkus-bot quarkus-bot bot added area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/platform Issues related to definition and interaction with Quarkus Platform labels Sep 23, 2023
@quarkus-bot
Copy link

quarkus-bot bot commented Sep 23, 2023

Thanks for your pull request!

The title of your pull request does not follow our editorial rules. Could you have a look?

  • title should preferably start with an uppercase character (if it makes sense!)
  • title should not contain an issue number (use Fix #1234 in the description instead)

This message is automatically generated by a bot.

@Sap004 Sap004 changed the title fixes #36109 Fixes incorrect redirection to dev-v1 Sep 23, 2023
@@ -255,7 +255,7 @@ <h1>You just made a Quarkus application.</h1>
<p>App configuration: <code>src/main/resources/{config.file-name}</code></p>
<p>Static assets: <code>src/main/resources/META-INF/resources/</code></p>
<p>Code: <code>{language.dir.code}</code></p>
<p>Dev UI V1: <a href="/q/dev-v1/">/q/dev-v1</a></p>
<p>Dev UI V1: <a href="/q/dev-ui/">/q/dev-v1</a></p>
Copy link
Contributor

Choose a reason for hiding this comment

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

V1 should be removed from the '<p>' as well

Copy link
Contributor Author

Choose a reason for hiding this comment

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

hi @manofthepeace ,
Removed V1 from

, requesting for approval

Copy link
Member

@phillip-kruger phillip-kruger left a comment

Choose a reason for hiding this comment

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

Thanks !

@phillip-kruger phillip-kruger added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Sep 25, 2023
@quarkus-bot

This comment has been minimized.

@@ -255,7 +255,7 @@ <h1>You just made a Quarkus application.</h1>
<p>App configuration: <code>src/main/resources/{config.file-name}</code></p>
<p>Static assets: <code>src/main/resources/META-INF/resources/</code></p>
<p>Code: <code>{language.dir.code}</code></p>
<p>Dev UI V1: <a href="/q/dev-v1/">/q/dev-v1</a></p>
<p>Dev UI: <a href="/q/dev-ui/">/q/dev-ui</a></p>
Copy link
Member

Choose a reason for hiding this comment

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

The Dev UI link is already present above. Let's drop this altogether. I will fine tune this and get this puppy home.

@gsmet gsmet force-pushed the bugfix/36109-welcome-page-error branch from 2347d5f to a95dccf Compare October 3, 2023 14:42
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.

This should be good to go now. Let's wait for CI and merge.

@gsmet gsmet changed the title Fixes incorrect redirection to dev-v1 Remove link to Dev UI v1 as it is now gone Oct 3, 2023
@quarkus-bot
Copy link

quarkus-bot bot commented Oct 3, 2023

✔️ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

@phillip-kruger phillip-kruger merged commit 677292f into quarkusio:main Oct 3, 2023
50 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.5 - main milestone Oct 3, 2023
@quarkus-bot quarkus-bot bot removed the triage/waiting-for-ci Ready to merge when CI successfully finishes label Oct 3, 2023
@gsmet gsmet modified the milestones: 3.5 - main, 3.4.2 Oct 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/platform Issues related to definition and interaction with Quarkus Platform kind/bugfix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Creating an app via code.quarkus.io still advertises dev-v1 in the welcome page
4 participants