Skip to content

Model separation of concerns #756

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.

Sign up for GitHub

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

Jump to bottom
Merged
mzieniukbw merged 3 commits into from
Jan 28, 2026
Merged

Model separation of concerns #756

mzieniukbw merged 3 commits into from
Jan 28, 2026

Conversation

@mzieniukbw
Copy link
Contributor

@mzieniukbw mzieniukbw commented Jan 23, 2026 •
edited
Loading

🎟️ Tracking

📔 Objective

  • Adds model separation of concerns into "Architecture" -> "Server Architecture" as separated page.
  • Moved existing CQS from "Server Architecture" into it's own page.
  • Retained the "Server Architecture" landing page and route.

📸 Screenshots

@mzieniukbw mzieniukbw requested a review from a team as a code owner January 23, 2026 16:59
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jan 23, 2026 •
edited
Loading

Deploying contributing-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 690eb14
Status: ✅  Deploy successful!
Preview URL: https://258bb2ae.contributing-docs.pages.dev
Branch Preview URL: https://models-separation-of-concern.contributing-docs.pages.dev

View logs

@github-actions github-actions bot added the adr label Jan 23, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 23, 2026 •
edited
Loading

Logo
Checkmarx One – Scan Summary & Details – f7040298-34b1-4045-9571-5d77a40fc612

Great job! No new security vulnerabilities introduced in this pull request

withinfocus
withinfocus reviewed Jan 26, 2026
View reviewed changes
@mzieniukbw mzieniukbw changed the title Models separation of concerns Model separation of concerns Jan 28, 2026
@mzieniukbw mzieniukbw merged commit 6c81c46 into main Jan 28, 2026
28 checks passed
@mzieniukbw mzieniukbw deleted the models-separation-of-concerns branch January 28, 2026 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@withinfocus withinfocus withinfocus approved these changes

Assignees

No one assigned

Labels

adr

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mzieniukbw @withinfocus