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

Reorganize docs and add automation to push them to the website #773

Open
derberg opened this issue Jun 28, 2023 · 4 comments · May be fixed by #1082
Open

Reorganize docs and add automation to push them to the website #773

derberg opened this issue Jun 28, 2023 · 4 comments · May be fixed by #1082
Labels

Comments

@derberg
Copy link
Member

derberg commented Jun 28, 2023

So we finally have this lovely stuff https://www.asyncapi.com/docs/community

And in this repo we have a bunch of documents that should be visible there on the website. And in future we need to add much more.

So the scope:

cc @thulieblack @alequetzalli @AceTheCreator

@quetzalliwrites
Copy link
Member

  • I suggest we move all website related docs to docs folder

Agreed!

  • documents that we want to display on the website need proper front matter settings, order and all that stuff. Probably links review?

Yes, we need to set up PRs and conduct a new editorial review since those changes will make those docs visible on the website docs.

I still don't like/approve of this workflow. Docs in repos where I am not specifically added as a maintainer are often merged without a proper review/editorial round. When merged, I still want docs in other repos to create a NEW PR in /website repo to enforce actual TW reviews. Many mistakes are merged this way because the writers submitting their docs PRs in other repos cannot review their content in a Netflify website preview link. It is not a good practice for core docs maintainers to not be aware/have the ability to review all docs PRs in diverse repos and for those docs to be able to be merged without proper docs maintainer reviews.

Copy link
Member Author

derberg commented Jun 29, 2023

Yes, we need to set up PRs and conduct a new editorial review since those changes will make those docs visible on the website docs.

and one document per PR to make it easier to iterate 😅

I still don't like/approve of this workflow. Docs in repos where I am not specifically added as a maintainer are often merged without a proper review/editorial round. When merged, I still want docs in other repos to create a NEW PR in /website repo to enforce actual TW reviews. Many mistakes are merged this way because the writers submitting their docs PRs in other repos cannot review their content in a Netflify website preview link. It is not a good practice for core docs maintainers to not be aware/have the ability to review all docs PRs in diverse repos and for those docs to be able to be merged without proper docs maintainer reviews.

luckily you are maintainer for community repo 😃 and yes, for website preview, we need an issue to be able to address it. I agree it would be nice.

Copy link

github-actions bot commented Feb 5, 2024

This issue 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 issue, add a comment with a detailed explanation.

There can be many reasons why some specific issue 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 issue forward. Connect with us through one of many communication channels we established here.

Thank you for your patience ❤️

Copy link

This issue 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 issue, add a comment with a detailed explanation.

There can be many reasons why some specific issue 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 issue forward. Connect with us through one of many communication channels we established here.

Thank you for your patience ❤️

@github-actions github-actions bot added the stale label Jul 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants
@derberg @quetzalliwrites and others