Skip to content

docs: remove the internal boundaries section - #3011

Merged
mfal merged 2 commits into
mainfrom
claude/remove-internal-boundaries-section-149fa9
Aug 31, 2026
Merged

docs: remove the internal boundaries section#3011
mfal merged 2 commits into
mainfrom
claude/remove-internal-boundaries-section-149fa9

Conversation

@Lisa18289

@Lisa18289 Lisa18289 commented Aug 31, 2026

Copy link
Copy Markdown
Member

Removes 02-foundations/04-internal/01-boundaries with its 19 live-code examples. We decided to drop the guideline here: it is outdated, and boundaries are documented directly in mStudio by now. It also addressed the internal mittwald team and described mStudio components rather than Flow ones, so it does not belong in the public styleguide. Its only inbound link — a sentence in the error handling pattern — is gone as well.

StaticModal existed only for that section's modal example. It drops with it, along with the LiveCodeEditor's component scope that exposed it, its CSS block and the global type declarations in both apps.

Docs-only, so nothing gets published.

🤖 Generated with Claude Code

The guideline targeted the internal mittwald team and described mStudio
components, not Flow ones. Its only reference — a link from the error
handling pattern — is gone too.

StaticModal existed solely for this section's modal example, so the
LiveCodeEditor's component scope, its CSS and the global type
declarations drop with it.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@Lisa18289 Lisa18289 self-assigned this Aug 31, 2026
…l-boundaries-section-149fa9

# Conflicts:
#	apps/docs/src/content/02-foundations/04-internal/01-boundaries/examples/modal-error.tsx
#	apps/docs/src/content/02-foundations/04-internal/01-boundaries/examples/modal-loading.tsx
@github-actions

Copy link
Copy Markdown
Contributor

Coverage Report for ./packages/components/

Status Category Percentage Covered / Total
🔵 Lines 76% 627 / 825
🔵 Statements 75.91% 643 / 847
🔵 Functions 77.95% 145 / 186
🔵 Branches 66.66% 298 / 447
File CoverageNo changed files found.
Generated in workflow #6439 for commit 526e619 by the Vitest Coverage Report Action

@Lisa18289
Lisa18289 marked this pull request as ready for review August 31, 2026 08:16
@Lisa18289
Lisa18289 requested a review from a team August 31, 2026 08:16
@github-actions

Copy link
Copy Markdown
Contributor

🚀 Preview Deployment

Preview environments are ready:

Type URL
docs pr-3011.docs.review.flow-components.de
storybook pr-3011.storybook.review.flow-components.de

Images:

  • docs: ghcr.io/mittwald/flow/docs:pr-3011
  • storybook: ghcr.io/mittwald/flow/storybook:pr-3011

@mfal
mfal merged commit 85bbc3b into main Aug 31, 2026
22 checks passed
@mfal
mfal deleted the claude/remove-internal-boundaries-section-149fa9 branch August 31, 2026 09:58
mfal added a commit that referenced this pull request Aug 31, 2026
#3011 removed the internal Boundaries section, which this branch had renamed.
The removal wins: the section and its examples are gone, and the link to it in
the Error Handling pattern goes with them.

That left `/foundations/internal` and `/foundations/internal/boundaries` in
`CONTENT_ORDER` pointing at nothing. `contentOrder.test.ts` named both, which
is the case it exists for — an order list in code going stale silently.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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.

2 participants