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 AsyncAPI Documents and change code snippet #2274

Conversation

AnimeshKumar923
Copy link
Contributor

@AnimeshKumar923 AnimeshKumar923 commented Oct 29, 2023

Description

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

Related issue(s)
fixes #1858

changes:

- Re-added the `AsyncAPI Documents` 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 b4e172e
🔍 Latest deploy log https://app.netlify.com/sites/shimmering-choux-eb0798/deploys/654ec559a0b46c0008ba82b6
😎 Deploy Preview https://deploy-preview-2274--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.

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 migrated AsyncAPI document. There is just one docu suggestion that you have to apply - just to make sure it is not about user application perspective but the perspective of the example app that this document describes

pages/docs/tutorials/getting-started/asyncapi-documents.md Outdated Show resolved Hide resolved
@AnimeshKumar923
Copy link
Contributor Author

@AnimeshKumar923 perfectly migrated AsyncAPI document.

Thank you! 🙏

There is just one docu suggestion that you have to apply - just to make sure it is not about user application perspective but the perspective of the example app that this document describes

applied suggestion with 24fdfd8

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

/rtm

@asyncapi-bot asyncapi-bot merged commit c913ca4 into asyncapi:next-major-spec Nov 11, 2023
9 checks passed
@AnimeshKumar923 AnimeshKumar923 deleted the adapt-AsyncAPI-Documents-tutorial-for-v3 branch November 11, 2023 07:11
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