-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
swapi.dev is down, use swapi.tech #45335
Conversation
Thanks for your pull request! Your pull request does not follow our editorial rules. Could you have a look?
This message is automatically generated by a bot. |
🙈 The PR is closed and the preview is expired. |
Status for workflow
|
@@ -135,7 +135,7 @@ them to the client as JSON using xref:rest-json.adoc[Jackson or JSON-B]: | |||
// Create a REST client to the Star Wars API | |||
WebClient client = WebClient.create(vertx, | |||
new WebClientOptions() | |||
.setDefaultHost("swapi.dev") | |||
.setDefaultHost("swapi.net") |
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.
The title of the PR says tech
and the quickstart change says tech
but here we have a net
? Is it expected?
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.
Good catch @gsmet, it should be .tech
as commit message says, I will create fix PR soon.
swapi.dev is down, use swapi.tech
swapi-graphql change: graphql/swapi-graphql@b340cf9
Related QS change: quarkusio/quarkus-quickstarts#1484