Skip to content

fix: promote reverse_domain_name to standalone type#260

Merged
wry-ry merged 1 commit intomainfrom
fix/reverse-domain-name-type
Mar 12, 2026
Merged

fix: promote reverse_domain_name to standalone type#260
wry-ry merged 1 commit intomainfrom
fix/reverse-domain-name-type

Conversation

@igrigorik
Copy link
Contributor

Fixes current mkdocs build failures.

Extract reverse_domain_name from ucp.json $defs into shopping/types/reverse_domain_name.json. Update all propertyNames refs in ucp.json to point to the standalone file directly.

The docs renderer generates page-local anchors for $defs entries (prefixed with the parent schema name), but standalone types get rendered on the reference page with a linkable section. The $defs indirection caused broken #ucp-reverse-domain-name anchors on checkout, cart, catalog, and reference pages.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas

  Extract reverse_domain_name from ucp.json $defs into
  shopping/types/reverse_domain_name.json. Update all propertyNames
  refs in ucp.json to point to the standalone file directly.

  The docs renderer generates page-local anchors for $defs entries
  (prefixed with the parent schema name), but standalone types get
  rendered on the reference page with a linkable section. The $defs
  indirection caused broken #ucp-reverse-domain-name anchors on
  checkout, cart, catalog, and reference pages.
@igrigorik igrigorik requested a review from wry-ry March 12, 2026 18:03
@igrigorik igrigorik requested review from a team as code owners March 12, 2026 18:03
Copy link

@knightlin-shopify knightlin-shopify left a comment

Choose a reason for hiding this comment

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

LGTM

@wry-ry wry-ry merged commit 08f565d into main Mar 12, 2026
7 checks passed
@wry-ry wry-ry deleted the fix/reverse-domain-name-type branch March 12, 2026 21:02
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.

4 participants