Skip to content

Document borderRadius for POS s-image (2026-07) - #4638

Merged
ajanth-u merged 3 commits into
2026-07from
au/pos-image-border-radius-2026-07
Aug 19, 2026
Merged

Document borderRadius for POS s-image (2026-07)#4638
ajanth-u merged 3 commits into
2026-07from
au/pos-image-border-radius-2026-07

Conversation

@ajanth-u

@ajanth-u ajanth-u commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

Problem

The POS s-image component supports borderRadius at runtime in API version 2026-07, but its public TypeScript contract and generated property reference omit the property. Valid extension code therefore fails type checking, and partners can't find the property in the reference.

Solution

Align the 2026-07 public contract with the existing runtime. Document the property's default and one-to-four-value flow-relative shorthand.

Assisted-By: devx/bd310388-3a75-41ab-8ca8-b7fa0e22c7e0
Assisted-By: devx/bd310388-3a75-41ab-8ca8-b7fa0e22c7e0
@github-actions github-actions Bot added the needs-rc-port PR against a stable branch awaiting forward-port to the current RC label Aug 19, 2026
@github-actions

Copy link
Copy Markdown
Contributor

This PR targets a stable release branch (2026-07). Once merged, the change typically also needs to be forward-ported to 2026-10-rc so it ships in the next release.

When you open the forward-port PR, include a line like this in its body so the needs-rc-port label gets removed automatically when that PR merges:

Forward-port of #4638

Accepted formats (comma-separated for multiple):

  • #4638
  • GH-4638
  • 4638
  • https://github.com/Shopify/ui-extensions/pull/4638

If a forward-port isn't needed (e.g., the change is stable-only), you can remove the needs-rc-port label manually.

@ajanth-u ajanth-u self-assigned this Aug 19, 2026
@ajanth-u
ajanth-u requested a review from prakashraman August 19, 2026 14:56
Assisted-By: devx/3102b0ac-6bc9-4efd-9954-fd7a371ab6f2
@ajanth-u ajanth-u changed the title Document borderRadius for POS s-image Document borderRadius for POS s-image (2026-07) Aug 19, 2026

@prakashraman prakashraman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Image

Tested locally. LGTM.

@ajanth-u
ajanth-u merged commit 3f5b0f4 into 2026-07 Aug 19, 2026
9 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

✓ Forward-ported in #4639.

@github-actions github-actions Bot removed the needs-rc-port PR against a stable branch awaiting forward-port to the current RC label Aug 19, 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.

2 participants