Skip to content

Commit 8828bfa

Browse files
committed
chore(changeset): app-shell patch for the default-form-only merge rule
Claude-Session: https://claude.ai/code/session_01FjqrwXPfSMkSfkKYDSRkN2 Co-authored-by: Claude <noreply@anthropic.com>
1 parent 69c5bc4 commit 8828bfa

1 file changed

Lines changed: 24 additions & 0 deletions

File tree

Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
---
2+
'@object-ui/app-shell': patch
3+
---
4+
5+
fix(app-shell): create and edit no longer render a container's first named form when it declares no default form
6+
7+
`@objectstack/spec` `ViewSchema` calls a view container's `form` its "Default form view" and
8+
`formViews` its "Additional named form views". The console's view merge (`applyViewItem` in
9+
`MetadataProvider`) made the FIRST form item to arrive an object's create/edit `.form` whenever no
10+
served form item carried `isDefault`. So a container with only named forms rendered its first named
11+
form as the create and edit form. In `examples/app-showcase`, `showcase_inquiry`'s only form is the
12+
public, anonymous contact-us form.
13+
14+
`.form` now comes only from a served form item that carries `isDefault`. Arrival order decides
15+
nothing: a default form arriving after a named one still wins, and a named form arriving after the
16+
default does not displace it. A container with no `form` leaves `.form` unset, and create and edit
17+
take the path an object with no form view takes: no curated sections, and the dialog lays out the
18+
object's own fields. A named form is reached only by its name, through `formViews`.
19+
20+
Servers that still stamp `isDefault` on a promoted named form see no change. The change takes effect
21+
with servers that serve a named-only container without a default form item (objectstack#21500). A
22+
stack-packaged container that reaches the console unexpanded is expanded by the bundled
23+
`@objectstack/spec`, which still flags its first named form as the default until that dependency
24+
carries the same fix.

0 commit comments

Comments
 (0)