Skip to content

Conversation

@thibaultzanini
Copy link
Collaborator

  • Fix spelling errors in registered-organization.mjson: "authorithy" → "authority" and "regsitrant" → "registrant"
  • Fix property name references in person-ui-descriptor.mjson: use actual schema property names (name.familyName, name.givenName)
  • Remove undefined "department" property reference

Copy link

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.

Pull request overview

This PR fixes spelling errors and incorrect property references in MJSON configuration files to ensure accurate schema definitions and UI descriptor expressions.

  • Fixed two spelling errors in registered-organization.mjson: "authorithy" → "authority" and "regsitrant" → "registrant"
  • Corrected property references in person-ui-descriptor.mjson to use actual schema property names (name.familyName, name.givenName)
  • Removed reference to undefined "department" property

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
data/model/party/registered-organization.mjson Corrects spelling errors in the description field for registrantOrganizationRegistrations property
data/model/party/person-ui-descriptor.mjson Updates nameExpression to use correct nested property paths (name.familyName, name.givenName) and removes invalid department reference from descriptionExpression

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

- Fix spelling errors in registered-organization.mjson: "authorithy" → "authority" and "regsitrant" → "registrant"
- Fix property name references in person-ui-descriptor.mjson: use actual schema property names (name.familyName, name.givenName)
- Use inherited "description" field from data-object for descriptionExpression
@thibaultzanini thibaultzanini force-pushed the claude/check-mjson-typos-0VW7r branch from 1731018 to 775d62f Compare January 7, 2026 17:59
@thibaultzanini
Copy link
Collaborator Author

LGTM

@thibaultzanini thibaultzanini self-assigned this Jan 8, 2026
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