website icon indicating copy to clipboard operation
website copied to clipboard

docs: add protocol concepts doc

Open nelsonmic opened this issue 3 years ago • 1 comments

Description

  • This PR adds a new protocol concept spec

Related issue(s)

Fixes #980

nelsonmic avatar Oct 11 '22 13:10 nelsonmic

Deploy Preview for asyncapi-website ready!

Built without sensitive environment variables

Name Link
Latest commit e10f5acd1835f26276b633377e1baf42d5c805f2
Latest deploy log https://app.netlify.com/sites/asyncapi-website/deploys/638d3c4b9b84cc0009f3f30a
Deploy Preview https://deploy-preview-1013--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 settings.

netlify[bot] avatar Oct 11 '22 13:10 netlify[bot]

hey @nelsonmic, we're close to the end of GSoD 😄... how is this doc going? When do you think we may see the first draft? ✨✨🙌🏽

quetzalliwrites avatar Oct 26 '22 01:10 quetzalliwrites

hey @nelsonmic, we're close to the end of GSoD 😄... how is this doc going? When do you think we may see the first draft? ✨✨🙌🏽

GSOD is not going to stop me from contributing 😄😊. I'll have this in by Monday.

nelsonmic avatar Oct 29 '22 20:10 nelsonmic

Hello @derberg, I'm having a problem with the diagram for this concept doc, please can you help.

First, confirm that the doc is technically correct, if it is then in my diagram I want to label the arrow pointing from producer to broker to consumer as protocol.

Here's what i have currently:

flowchart LR
    a[Producer]-- Message 1 --->c[(Broker)]
    c-- Message 1 ---> b[Consumer]

nelsonmic avatar Oct 30 '22 17:10 nelsonmic

⚡️ Lighthouse report for the changes in this PR:

Category Score
🟠 Performance 58
🟠 Accessibility 88
🟠 Best practices 83
🟢 SEO 100
🔴 PWA 30

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

github-actions[bot] avatar Oct 30 '22 17:10 github-actions[bot]

Hello @alequetzalli, good day! I just fixed the issue, please can you have a look at it.

nelsonmic avatar Nov 07 '22 13:11 nelsonmic

I think this looks good, but we should def see if @derberg agrees it's ready to merge :)

quetzalliwrites avatar Nov 07 '22 15:11 quetzalliwrites

hey @nelsonmic, please add to your PR:

  1. The actual diagram syntax markdown, even if you think it has mistakes. I know you put it in some comment, but now it's lost and cannot be easily found to review. In addition, Lukasz will be able to easily add a commit suggestion to you if you actually have that diagram as part of your PR, making it easier to fix any issues the diagram has.
  2. Prev/Next buttons (I'm putting my application doc before yours)

quetzalliwrites avatar Nov 07 '22 21:11 quetzalliwrites

hey @nelsonmic, please add to your PR:

  1. The actual diagram syntax markdown, even if you think it has mistakes. I know you put it in some comment, but now it's lost and cannot be easily found to review. In addition, Lukasz will be able to easily add a commit suggestion to you if you actually have that diagram as part of your PR, making it easier to fix any issues the diagram has.
  2. Prev/Next buttons (I'm putting my application doc before yours)

Hey hey 😌, thank you for the feedback. I'll get right to it.

nelsonmic avatar Nov 08 '22 12:11 nelsonmic

Heyo @nelsonmic, looks like you got some conflicts to solve. 😄✌🏽

Screen Shot 2022-11-08 at 2 51 44 PM

quetzalliwrites avatar Nov 08 '22 22:11 quetzalliwrites

Hey @nelsonmic! 😄 How's this PR going? do you understand all the feedback @derberg left you?

Let's take care of this doc before we create a NEW doc for bindings.

quetzalliwrites avatar Nov 16 '22 00:11 quetzalliwrites

Hey @derberg please can you review the diagram and definition again, thank you!

nelsonmic avatar Nov 16 '22 15:11 nelsonmic

Hey @nelsonmic! 😄 How's this PR going? do you understand all the feedback @derberg left you?

Let's take care of this doc before we create a NEW doc for bindings.

Hi @alequetzalli, I just pushed a commit to address @derberg suggestions, I'm waiting on his feedback

nelsonmic avatar Nov 16 '22 15:11 nelsonmic

Hey @alequetzalli, I have fixed everything that @derberg suggested. This PR is ready for an editorial review from you, thanks.

nelsonmic avatar Nov 27 '22 01:11 nelsonmic

I confirm that technically all accurate 👍🏼 @alequetzalli only your review is needed

derberg avatar Dec 01 '22 12:12 derberg

Side comment. I think Open ai just released one of the best tools that can be used as technical input for docs.

example answer to question "what is protocol"

A protocol is a set of rules and guidelines that define the communication and interactions between different devices or systems. Protocols ensure that the devices or systems can communicate and exchange information in a consistent and efficient manner. Examples of protocols include the HTTP protocol for web communication, the TCP/IP protocol for internet communication, and the protocols used in wireless networks.

🤯 https://chat.openai.com

derberg avatar Dec 02 '22 08:12 derberg

@derberg you forgot to approve this PR 😜 merging will be blocked until you do it since you requested changes before

quetzalliwrites avatar Dec 03 '22 00:12 quetzalliwrites

Yay! 😁 congratulations @nelsonmic, great work!

@derberg can you please merge this AND the application PR together now? #992

Thank you, thank you, thank you 😁. Bindings concept next

nelsonmic avatar Dec 05 '22 00:12 nelsonmic

@alequetzalli oh, I thought it is still pending editorial review, this is why I did not accept as I was sure there will be some additional commits that will dismiss my review.

derberg avatar Dec 05 '22 11:12 derberg

@derberg Yo you forgot to merge both PRs 😛

quetzalliwrites avatar Dec 05 '22 17:12 quetzalliwrites

/rtm

derberg avatar Dec 06 '22 07:12 derberg