Skip to content

docs: enhance layered-project-dependencies diagrams#25461

Merged
EngincanV merged 5 commits into
rel-10.4from
maliming/issue-25457-layered-diagrams
May 26, 2026
Merged

docs: enhance layered-project-dependencies diagrams#25461
EngincanV merged 5 commits into
rel-10.4from
maliming/issue-25457-layered-diagrams

Conversation

@maliming
Copy link
Copy Markdown
Member

@maliming maliming commented May 22, 2026

Fixes #25457.

Add the missing .Application -> .Domain arrow to all 4 diagrams. The module diagram also adds .MongoDB and replaces the old .Web -> .HttpApi edge with .Web -> .Application.Contracts (the truthful edge per the template's csproj and [DependsOn]).

- Add `.Application -> .Domain` arrow to all 4 diagrams
- Module diagram: remove `.Web` node (module template's `.Web` is the
  module's own UI layer per `[DependsOn]` and csproj, not a standalone
  host that would reference `.HttpApi`) and add `.MongoDB`
Copilot AI review requested due to automatic review settings May 22, 2026 02:38
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

@github-actions
Copy link
Copy Markdown
Contributor

Images automagically compressed by Calibre's image-actions

Compression reduced images by 66.9%, saving 49.1 KB.

Filename Before After Improvement Visual comparison
docs/en/images/layered-project-dependencies-blazor-wasm.png 19.4 KB 6.4 KB 66.8% View diff
docs/en/images/layered-project-dependencies-blazor-server.png 18.1 KB 5.9 KB 67.1% View diff
docs/en/images/layered-project-dependencies.png 18.0 KB 5.9 KB 67.4% View diff
docs/en/images/layered-project-dependencies-module.png 17.9 KB 6.0 KB 66.4% View diff

@maliming maliming added this to the 10.4-patch milestone May 22, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Images automagically compressed by Calibre's image-actions

Compression reduced images by 66.8%, saving 50.3 KB.

Filename Before After Improvement Visual comparison
docs/en/images/layered-project-dependencies-module.png 19.9 KB 6.8 KB 65.8% View diff
docs/en/images/layered-project-dependencies-blazor-wasm.png 19.4 KB 6.4 KB 66.8% View diff
docs/en/images/layered-project-dependencies-blazor-server.png 18.1 KB 5.9 KB 67.1% View diff
docs/en/images/layered-project-dependencies.png 18.0 KB 5.9 KB 67.4% View diff

@EngincanV EngincanV merged commit 61c00bf into rel-10.4 May 26, 2026
1 check passed
@EngincanV EngincanV deleted the maliming/issue-25457-layered-diagrams branch May 26, 2026 06:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants