Skip to content

support special URL paths such as .well-known.#708

Open
hansbak wants to merge 1 commit intomoqui:masterfrom
growerp:extend-xsd-url-check
Open

support special URL paths such as .well-known.#708
hansbak wants to merge 1 commit intomoqui:masterfrom
growerp:extend-xsd-url-check

Conversation

@hansbak
Copy link
Copy Markdown
Contributor

@hansbak hansbak commented Apr 3, 2026

No description provided.

Copilot AI review requested due to automatic review settings April 3, 2026 03:18
Copy link
Copy Markdown

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 updates the XML Screen XSD to allow transition names that support special URL paths (eg .well-known) by expanding the allowed character pattern for the transition/@name attribute.

Changes:

  • Updated transition/@name documentation to mention leading-dot path support (eg .well-known).
  • Added an inline XSD pattern restriction intended to allow . and - in transition names.

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

Comment on lines +329 to +333
<xs:simpleType>
<xs:restriction base="xs:token">
<xs:pattern value="[\-\.a-zA-Z][\-_\.a-zA-Z0-9]*" />
</xs:restriction>
</xs:simpleType>
Copy link

Copilot AI Apr 3, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This inline xs:simpleType makes the schema invalid because the attribute already declares type="name-package" (XSD attributes cannot have both type and an anonymous simpleType). Consider defining a dedicated named simpleType (eg for transition path names) and referencing it from the name attribute (and other places that accept transition names), or remove the type attribute and derive the restriction from name-package/xs:string instead.

Copilot uses AI. Check for mistakes.
Comment on lines 323 to +327

The transition name is used in link and other elements in place of URLs when going to another
screen within this application. The transition name will appear briefly as the URL before the
redirect is done for the transition response.
</xs:documentation></xs:annotation>
redirect is done for the transition response. A name may also start with a
dot to support special URL paths such as .well-known. </xs:documentation>
Copy link

Copilot AI Apr 3, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The documentation in this block still emphasizes camelCase/lower-case naming, but the change is introducing support for leading-dot URL paths (eg .well-known) and the new pattern also broadens allowed characters. Please adjust the wording to accurately describe the allowed formats/exceptions so it matches the schema validation.

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure what need to changed? already describing with the .well-known. example?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants