- Issue created by @Kristen Pol
- πΊπΈUnited States Kristen Pol Santa Cruz, CA, USA
Moving this to the ai project because that repo will be used
- πΊπΈUnited States Kristen Pol Santa Cruz, CA, USA
kristen pol β changed the visibility of the branch 3534360-document-new-ai to hidden.
- πΊπΈUnited States Kristen Pol Santa Cruz, CA, USA
Ignore the hidden branch... pushed code for a different issue in there by accident.
- πΊπΈUnited States Kristen Pol Santa Cruz, CA, USA
I made a first pass, but I need to review it as I'm sure there are some mistakes.
- πΊπΈUnited States Kristen Pol Santa Cruz, CA, USA
Pinged Marcus+Vall+Jamie+Christoph for review
- π©πͺGermany breidert
I did an in depth review of the proposed documentation change.
I really like the quality and the clarify of the new "Issue guidelines". However, I find them a bit long and fear contributors might not read them. But after spending some time thinking about it, I could not suggest improvements or come up with a better way to document issues.
I only suggested some small text changes for which I felt they made the text more neutral. I left the suggestions as comments in the MR.
However, when my suggestions are accepted or declined the issue can be set to RTBC from my point of view.
- πΊπΈUnited States Kristen Pol Santa Cruz, CA, USA
Thanks for the feedback. I'll be updating this shortly.
- πΊπΈUnited States Kristen Pol Santa Cruz, CA, USA
Updated and set to RTBC per #11. I'll get this merged.
- πΊπΈUnited States Kristen Pol Santa Cruz, CA, USA
Merged... not sure if this one makes sense to backport so I'll check with Marcus.
- π©πͺGermany marcus_johansson
kristen pol β credited marcus_johansson β .
- πΊπΈUnited States Kristen Pol Santa Cruz, CA, USA
Marcus said to backport it and also:
Also, I think we should the issues guidelines in the left menu under Contribute? This can be done in mkdocs.yml in the root of the project.
- πΊπΈUnited States Kristen Pol Santa Cruz, CA, USA
Moving back to needs work for Marcus' suggestions
- πΊπΈUnited States Kristen Pol Santa Cruz, CA, USA
Created follow-up issue: π AI module documentation has warnings when running mkdocs serve Active
- Merge request !793Add new doc page to menu and add table of contents. β (Merged) created by Kristen Pol
- πΊπΈUnited States Kristen Pol Santa Cruz, CA, USA
kristen pol β changed the visibility of the branch 3534360-issue-workflow-docs-1.1.x to hidden.
- Merge request !795Resolve #3534360 "Issue workflow docs 1.1.x v2" β (Merged) created by Kristen Pol
- πΊπΈUnited States Kristen Pol Santa Cruz, CA, USA
Merged changes to 1.2.x. The same changes for 1.1.x are being merged.
-
kristen pol β
committed 2c4de3bf on 1.1.x
Resolve #3534360 "Issue workflow docs 1.1.x v2"
-
kristen pol β
committed 2c4de3bf on 1.1.x
- πΊπΈUnited States Kristen Pol Santa Cruz, CA, USA
Everything is merged. Not sure when/how:
https://project.pages.drupalcode.org/ai/1.1.x/contribute/documentation/
gets updated though... I'll check
- πΊπΈUnited States Kristen Pol Santa Cruz, CA, USA
Marcus fixed the syncing