Skip to content

Conversation

@CollierCZ
Copy link
Contributor

Why

Closes #2319

What's changed

@CollierCZ CollierCZ added the docs rework For issues about updating the docs to the new style and content model (see the project for more) label Aug 31, 2022
@CollierCZ CollierCZ self-assigned this Aug 31, 2022
@CollierCZ CollierCZ requested a review from trolologuy August 31, 2022 16:25
Copy link
Contributor

@trolologuy trolologuy left a comment

Choose a reason for hiding this comment

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

I really like how plantuml has a much better look and feel and integrates better than other tools like mermaid.js (but is a tad harder to maintain).
It's a welcome addition and really helps to understand the flow and the logic!

There are a few typos. Other than that, the split / logic is much better than it was. Great PR!

@CollierCZ CollierCZ requested a review from trolologuy September 2, 2022 07:24
Copy link
Contributor

@trolologuy trolologuy left a comment

Choose a reason for hiding this comment

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

  • Final built pages are working as expected.
  • The information is correct.
  • The information is comprehensive.
  • The information is concise.
  • The information is clear.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs rework For issues about updating the docs to the new style and content model (see the project for more)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

📝 Update YAML overview

3 participants