Conversation
0b60452 to
12584ef
Compare
Add a section that discusses the AEP tooling ecosystem. This will better improve the discoverability of the tools.
12584ef to
c94d54c
Compare
|
here's a PR to add mermaid support to site-generator: aep-dev/site-generator#59. We should merge that first. |
rambleraptor
left a comment
There was a problem hiding this comment.
The wording on this looks great. Do we have any control over styling? Some of the nodes are cut off in your screenshot + it would be nice to differentiate them based on color (planned, exists, external)
I think the cutoff will be fixed during the render - the screenshot was in vscode, not the new site-generator. Re the colors I don't think we have much control, and we also should be careful about being colorblind friendly. Let's push it and then make incremental changes? |
Add a section that discusses the AEP tooling ecosystem. This will better improve the discoverability of the tools.
The site-generator does not currently support mermaid, but I think it'd be great if it did and we merge this in rather that switch to graphviz, which is missing highlighting of links.
Example render in my editor:
🍱 Types of changes
What types of changes does your code introduce to AEP? Put an
xin the boxesthat apply
📋 Your checklist for this pull request
Please review the AEP Style and Guidance for
contributing to this repository.
General
references AEPs
correctly.
(usually
prettier -w .)💝 Thank you!