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

docs: split hello world tutorial into two (send and receive) #2981

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

quetzalliwrites
Copy link
Member

Resolves #993 and comment by @derberg via #993 (comment).

@quetzalliwrites quetzalliwrites added 📑 docs area/docs Specify what technical area given issue relates to. Its goal is to ease filtering good first issues. labels May 21, 2024
@quetzalliwrites quetzalliwrites self-assigned this May 21, 2024
Copy link

netlify bot commented May 21, 2024

Deploy Preview for asyncapi-website ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 606e55d
🔍 Latest deploy log https://app.netlify.com/sites/asyncapi-website/deploys/669e1f15fc0fa00008016733
😎 Deploy Preview https://deploy-preview-2981--asyncapi-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@asyncapi-bot
Copy link
Contributor

asyncapi-bot commented May 21, 2024

⚡️ Lighthouse report for the changes in this PR:

Category Score
🔴 Performance 36
🟢 Accessibility 98
🟢 Best practices 92
🟢 SEO 100
🟠 PWA 56

Lighthouse ran on https://deploy-preview-2981--asyncapi-website.netlify.app/

Copy link
Member

Choose a reason for hiding this comment

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

Kindly change the folder for these files, as inside markdown.

Copy link
Member Author

Choose a reason for hiding this comment

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

Got it, thanks for the reminder!

@derberg
Copy link
Member

derberg commented Jun 18, 2024

what happens with https://github.com/asyncapi/website/blob/master/markdown/docs/tutorials/getting-started/hello-world.md tutorial? shouldn't one of new once replace it with same filename but different title (asking because of dead links)


The `operations` section is where you describe what the application is doing. Each operation has a unique identifier for example, `sendHello`.

In the above example, you see that the `Hello world application` is a producer sending the `sayHelloMessage` message from the `hello` channel. In other words, you can say that the `Hello world application` publishes the `hello` topic to `send` the `sayHelloMessage` message. That AsyncAPI document describes what the `Hello world application` is doing, not what others can do with it.
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
In the above example, you see that the `Hello world application` is a producer sending the `sayHelloMessage` message from the `hello` channel. In other words, you can say that the `Hello world application` publishes the `hello` topic to `send` the `sayHelloMessage` message. That AsyncAPI document describes what the `Hello world application` is doing, not what others can do with it.
In the above example, you see that the `Hello world application` is a producer sending the `sayHelloMessage` message to the `hello` channel. In other words, you can say that the `Hello world application` `send` the `sayHelloMessage` message to the `hello` topic. That AsyncAPI document describes what the `Hello world application` is doing, not what others can do with it.

@@ -0,0 +1,120 @@
---
title: "Receive 'Hello world' message"
Copy link
Member

Choose a reason for hiding this comment

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

the only difference in this document is receive operation and there is a lot of repetition - is it intended? especially that app name is the same Hello world application. Shouldn't it be done like other tuturials for example for Kafka, where first tutorial would be about send message, and the one about receiving would be "extending" the send-one, where you extend the document with additional receive operation (of course still explaining these can be separate)

Copy link
Member Author

quetzalliwrites commented Jun 21, 2024

There would no longer be that link.

Instead, we would have two new links that help differentiate the two hello world tutorials in this section.

As for dead links, I will make sure to check for any in this PR too now that it's being reviewed.

Copy link
Member

derberg commented Jun 24, 2024

then we need redirect from old link to a new one, for external resources that link to the current document, to not cause dead links

@quetzalliwrites
Copy link
Member Author

Oh, right! I forgot, thanks for reminding me. smh 🤦‍♀️

Can you remind/tell me where I should be adding the redirect links? I remember before I did it for the spec docs, but this is the website repo and I'm unsure. 🫤

@derberg

Copy link
Member

derberg commented Jun 25, 2024

@sambhavgupta0705
Copy link
Member

/update

@akshatnema
Copy link
Member

@quetzalliwrites @derberg Any update on this PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/docs Specify what technical area given issue relates to. Its goal is to ease filtering good first issues. 📑 docs
Projects
Status: Community PR under Review 🧐
Development

Successfully merging this pull request may close these issues.

[📑 Docs]: add new tutorial Hello World for producer application
6 participants