-
Notifications
You must be signed in to change notification settings - Fork 130
docs: add academy debugging guide using layer 5 cloud events #731
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
Conversation
✅ Deploy Preview for bejewelled-pegasus-b0ce81 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Thank you for your contribution! Add it as an agenda item to the meeting minutes, if you would :) |
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.
Hi @kirti763, thanks for this.
Would it be possible to convert this section into a collapsible section? Since these are specific issues that not all users will face, I think presenting them this way would lead to a cleaner and more streamlined user experience.
Additionally, I believe the current location (/extending-the-academy) isn't the most appropriate. Placing this information in the FAQ section of creating-your-learning-path would be a better fit. What do you think?
Totally agree with you, @zihanKuang |
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.
Hi @kirti763, there's a merge conflict.
The FAQ format in academy/creating-your-learning-path/index.md
has been updated, so you'll need to update your branch.
Signed-off-by: kirti763 <kg4180553@gmail.com>
638e932
to
f5d7384
Compare
done, @zihanKuang |
@kirti763 Did you accidentally copy an extra file?👀 |
Signed-off-by: Zihan Kuang <127078886+zihanKuang@users.noreply.github.com>
Notes for Reviewers
This PR fixes #726
Signed commits