Skip to content

docs(intent): personalReadOnly is declarable on a composition edge too (#7340) - #243

Merged
delchev merged 1 commit into
masterfrom
child-personal-readonly
Sep 12, 2026
Merged

docs(intent): personalReadOnly is declarable on a composition edge too (#7340)#243
delchev merged 1 commit into
masterfrom
child-personal-readonly

Conversation

@delchev

@delchev delchev commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

Documents eclipse-dirigible/dirigible#7354.

The DSL reference said personalReadOnly goes with personal: true and that composition children inherit it through the parent. That coupling left the commonest personal-surface shape unexpressible: a header the owner authors whose lines an engine writes (a leave request whose day rows the approval flow charges against an entitlement) - the child was writable exactly when the header was, so the generated child MyController accepted a create from the owner's own draft against any entitlement in the tenant.

The paragraph now covers both ends: alongside personal: true it closes the declaring entity's own surface, and on a composition relation it closes only that child's inherited surface while the parent stays writable - with the effect on the generated pages and the cases where the key is refused.

npm run docs:build passes.

🤖 Generated with Claude Code

…o (#7340)

The key closed a whole personal surface or nothing, and a composition child
cannot carry `personal:` at all, so a header the owner authors whose lines an
engine writes had no expressible shape - the child was writable exactly when
its parent was. On the composition relation it closes only the child's
inherited surface: the child's MyController answers 403 and the parent's
my/document page drops the items Add, the row actions and that child panel's
Add, while the header keeps Save and Delete.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@delchev
delchev merged commit c97fb33 into master Sep 12, 2026
@delchev
delchev deleted the child-personal-readonly branch September 12, 2026 05:44
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.

1 participant