Skip to content

CEXT-6413: Migrate product / custom-mass-action to Admin UI V2#61

Open
jcuerdo wants to merge 3 commits into
feature/admin-ui-sdk-v2-samplesfrom
CEXT-6413
Open

CEXT-6413: Migrate product / custom-mass-action to Admin UI V2#61
jcuerdo wants to merge 3 commits into
feature/admin-ui-sdk-v2-samplesfrom
CEXT-6413

Conversation

@jcuerdo

@jcuerdo jcuerdo commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

jcuerdo added 3 commits July 1, 2026 21:45
Align with the canonical V2 README structure (What you get / How it
works / Setup with associate-and-install step) instead of the copied
V1 content.

@iivvaannxx iivvaannxx left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Missing all the migration to the new helpers of lib-admin-ui, did you run the scaffold?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Was this file created? How come it doesn't appear like the others as renamed only?

Comment on lines +19 to +20
async function main(params) {
const { gridType, ids } = parseMassActionRequest(params)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Your suggestion can be applied here: adobe/aio-commerce-sdk#543 (comment)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

what's this

@obarcelonap

Copy link
Copy Markdown
Member

see my comment in https://jira.corp.adobe.com/browse/CEXT-6413
the idea is to have self contained examples, both v1 and v2 that has banner notifications

@@ -1 +1 @@
$schema: http://json-schema.org/draft-07/schema

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Remove $schema,$id and categories since it is not needed, not sure why the agent adds this.
I did the same for all the v2 examples, see 66b750f

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