Skip to content

feat[plugins][commerceLayer]: ENG-11943 Display SKU/Bundle Code Instead of ID in Commerce Layer Plugin#4505

Merged
AishwaryaParab merged 1 commit intomainfrom
ENG-11943_display_skuCode_instead_of_id
Apr 7, 2026
Merged

feat[plugins][commerceLayer]: ENG-11943 Display SKU/Bundle Code Instead of ID in Commerce Layer Plugin#4505
AishwaryaParab merged 1 commit intomainfrom
ENG-11943_display_skuCode_instead_of_id

Conversation

@AishwaryaParab
Copy link
Copy Markdown
Contributor

@AishwaryaParab AishwaryaParab commented Apr 7, 2026

Description

When a SKU or Bundle is selected in the Commerce Layer plugin, the field label displays the internal Commerce Layer ID (e.g., nPwySKzvQM) instead of the human-readable SKU/Bundle code (e.g., 833600101). This makes it difficult for editors to identify which SKU or Bundle is selected at a glance.

The ResourcePreviewCell component in plugin-tools was using resource.id (the internal ID) for the label. We're now using resource.handle to display the SKU Code.

Screenshot
SKU created in Commerce Layer:
Screenshot 2026-04-07 at 11 38 05 AM

BEFORE:
Screenshot 2026-04-07 at 11 37 52 AM

AFTER:
Screenshot 2026-04-07 at 11 30 47 AM


Note

Low Risk
Low risk UI-only change that affects how selected resources are labeled in the picker/preview, with a safe fallback to id when handle is missing.

Overview
Updates ResourcePreviewCell in ResourcesPicker.tsx to display a resource’s human-friendly handle (e.g., SKU/bundle code) alongside the title instead of the internal id, falling back to id when no handle is available.

Reviewed by Cursor Bugbot for commit 88e1e32. Bugbot is set up for automated code reviews on this repo. Configure here.

@AishwaryaParab AishwaryaParab requested review from a team and anaghav2023 and removed request for a team April 7, 2026 06:09
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Apr 7, 2026

⚠️ No Changeset found

Latest commit: 88e1e32

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@nx-cloud
Copy link
Copy Markdown

nx-cloud Bot commented Apr 7, 2026

View your CI Pipeline Execution ↗ for commit 88e1e32

Command Status Duration Result
nx test @e2e/hydrogen ✅ Succeeded 5m 18s View ↗
nx test @e2e/qwik-city ✅ Succeeded 7m 53s View ↗
nx test @e2e/nextjs-sdk-next-app ✅ Succeeded 7m 9s View ↗
nx test @e2e/angular-17 ✅ Succeeded 6m 29s View ↗
nx test @e2e/react-sdk-next-14-app ✅ Succeeded 5m 15s View ↗
nx test @e2e/react-sdk-next-15-app ✅ Succeeded 5m 18s View ↗
nx test @e2e/nuxt ✅ Succeeded 5m 26s View ↗
nx test @e2e/angular-17-ssr ✅ Succeeded 5m 22s View ↗
Additional runs (38) ✅ Succeeded ... View ↗

☁️ Nx Cloud last updated this comment at 2026-04-07 06:29:09 UTC

@AishwaryaParab AishwaryaParab merged commit 05180e9 into main Apr 7, 2026
48 checks passed
@AishwaryaParab AishwaryaParab deleted the ENG-11943_display_skuCode_instead_of_id branch April 7, 2026 06:36
AishwaryaParab added a commit that referenced this pull request Apr 7, 2026
…in-tools and commercelayer plugins (#4506)

## Description

Bumping up the versions for plugin-tools and commercelayer plugins. 

Related PR (merged): #4505

_Screenshot_
N/A

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Low Risk**
> Low risk version/lockfile bumps only; main risk is
publishing/consuming the new `@builder.io/plugin-tools@0.0.9` package in
`plugin-commercelayer` without runtime code changes.
> 
> **Overview**
> Bumps `@builder.io/plugin-tools` from `0.0.8` to `0.0.9` (package +
lockfile).
> 
> Bumps `@builder.io/plugin-commercelayer` from `0.1.4` to `0.1.5` and
updates its dependency/lockfile to consume
`@builder.io/plugin-tools@0.0.9` (including updated tarball integrity
metadata).
> 
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
0bf5f74. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->
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