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: re-add Servers tutorial and change code snippet #2275

Conversation

AnimeshKumar923
Copy link
Contributor

@AnimeshKumar923 AnimeshKumar923 commented Oct 29, 2023

Description

  • Re-added the Servers tutorial in the next-major-spec branch.
  • Changed the code snippet to match v3.

Related issue(s)

Fixes #1859

changes:

- Re-added the `Severs tutorial` in the next-major-spec branch.
- Changed the code snippet to match v3.
@netlify
Copy link

netlify bot commented Oct 29, 2023

Deploy Preview for shimmering-choux-eb0798 ready!

Name Link
🔨 Latest commit fcf0eb9
🔍 Latest deploy log https://app.netlify.com/sites/shimmering-choux-eb0798/deploys/654ea71c39a8440008b795a0
😎 Deploy Preview https://deploy-preview-2275--shimmering-choux-eb0798.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.

@AnimeshKumar923 AnimeshKumar923 changed the title re-add Servers tutorial and change code snippet docs: re-add Servers tutorial and change code snippet Oct 29, 2023
derberg
derberg previously approved these changes Nov 8, 2023
Copy link
Member

@derberg derberg left a comment

Choose a reason for hiding this comment

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

@AnimeshKumar923 perfectly updated! 👏🏼

@alequetzalli you can approve and merge. Only example had to be changed. Animesh re-added existing tutorial and only modified the example - text unchanged

Screenshot 2023-11-08 at 17 09 21

@AnimeshKumar923
Copy link
Contributor Author

AnimeshKumar923 commented Nov 8, 2023

@AnimeshKumar923 perfectly updated! 👏🏼

Thank you! My pleasure Sir! salute-emoji

derberg
derberg previously approved these changes Nov 9, 2023
Copy link
Member

@quetzalliwrites quetzalliwrites left a comment

Choose a reason for hiding this comment

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

@derberg, i think we need you to hit approve again? yes? or no? here I go and see what happens 😂

@quetzalliwrites
Copy link
Member

/rtm

@asyncapi-bot asyncapi-bot merged commit 16f4e1f into asyncapi:next-major-spec Nov 10, 2023
10 checks passed
@quetzalliwrites
Copy link
Member

oh nvm this one didnt re request a review for some reason O_O that makes no sense to me, but hey, it's merged!

@AnimeshKumar923 AnimeshKumar923 deleted the adapt-Servers-tutorial-for-v3 branch November 11, 2023 07:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants