-
-
Notifications
You must be signed in to change notification settings - Fork 98
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: added table of contents and introduction document for Studio tool #553
Conversation
✅ Deploy Preview for modest-rosalind-098b67 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Welcome to AsyncAPI. Thanks a lot for creating your first pull request. Please check out our contributors guide useful for opening a pull request.
Keep in mind there are also other channels you can use to interact with AsyncAPI community. For more details check out this issue.
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.
Hey @alequetzalli, @derberg , @Florence-Njeri,
I have added Table of Contents
and Introduction
document in this PR. (I used Generator documentation for guidance)
For Usage
, I was unable to think of what content to start with.
Usage is not possible without explaining the Navigation Panel. What if I am not left with unique content for each document.
lol don't freak out yet @Jagrutiti ... this to me seems like a possibility for removing |
Co-authored-by: Alejandra Quetzalli <[email protected]>
Co-authored-by: Alejandra Quetzalli <[email protected]>
Co-authored-by: Alejandra Quetzalli <[email protected]>
Co-authored-by: Alejandra Quetzalli <[email protected]>
Co-authored-by: Alejandra Quetzalli <[email protected]>
…into doc/introduction
@alequetzalli @Jagrutiti Sorry for such a delay! Yes, we can now remove @alequetzalli Should we merge that in this shape? |
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.
I added some suggestions :) Great work @Jagrutiti !
/rtm |
Quality Gate passedIssues Measures |
@Amzani did you accept my commmit suggestions? It doesn't look like it, thus blocking the merge from happening. :) |
Co-authored-by: Quetzalli <[email protected]>
Co-authored-by: Quetzalli <[email protected]>
Co-authored-by: Quetzalli <[email protected]>
Co-authored-by: Quetzalli <[email protected]>
Co-authored-by: Quetzalli <[email protected]>
Co-authored-by: Quetzalli <[email protected]>
Co-authored-by: Quetzalli <[email protected]>
Co-authored-by: Quetzalli <[email protected]>
Co-authored-by: Quetzalli <[email protected]>
Co-authored-by: Quetzalli <[email protected]>
Co-authored-by: Quetzalli <[email protected]>
Co-authored-by: Quetzalli <[email protected]>
Co-authored-by: Quetzalli <[email protected]>
Co-authored-by: Quetzalli <[email protected]>
Quality Gate passedIssues Measures |
@magicmatatjahu can't merge this PR without your approval. can you take a look? |
This pull request has been automatically marked as stale because it has not had recent activity 😴 It will be closed in 120 days if no further activity occurs. To unstale this pull request, add a comment with detailed explanation. There can be many reasons why some specific pull request has no activity. The most probable cause is lack of time, not lack of interest. AsyncAPI Initiative is a Linux Foundation project not owned by a single for-profit company. It is a community-driven initiative ruled under open governance model. Let us figure out together how to push this pull request forward. Connect with us through one of many communication channels we established here. Thank you for your patience ❤️ |
Description
Added the following document for Studio tool for #552