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: add index for 100 level learning path #3

Merged
merged 4 commits into from
Dec 16, 2023

Conversation

mhmohona
Copy link
Contributor

Description

Added index file for 100 level introductory learning path.

Related issue(s)
asyncapi/website#1520

Copy link

@github-actions github-actions bot left a 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.

@mhmohona mhmohona changed the title Add index for 100 level learning path docs: add index for 100 level learning path Nov 28, 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.

I think this is correct... thoughts, @derberg? 😸

@TRohit20
Copy link

TRohit20 commented Dec 5, 2023

@alequetzalli Need to DM Lukasz to get him to check on this. I remember he said it's best to do so as his notifications are flooded.

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.

one change related to CLI version should be applied

100-level-AsyncAPI-Intro/setup_node.sh Show resolved Hide resolved
100-level-AsyncAPI-Intro/setup_node.sh Show resolved Hide resolved
Copy link

@TRohit20 TRohit20 left a comment

Choose a reason for hiding this comment

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

Suggested the change to add the latest CLI version based on Lukaz's suggestion

@quetzalliwrites
Copy link
Member

/rtm

@asyncapi-bot asyncapi-bot merged commit c5fc0a5 into asyncapi:master Dec 16, 2023
6 checks passed
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.

5 participants