Skip to content

add intro reference to the accs editor function#14

Merged
dshevtsov merged 5 commits intomainfrom
jh_accs-intro
Apr 20, 2026
Merged

add intro reference to the accs editor function#14
dshevtsov merged 5 commits intomainfrom
jh_accs-intro

Conversation

@jhadobe
Copy link
Copy Markdown
Collaborator

@jhadobe jhadobe commented Apr 14, 2026

This PR:

  • Adds an intro reference to an _includes file for ACCS schemas.

  • Adds an override to change the title to "Adobe Commerce as a Cloud Service"

  • Adds a prompt to update the version name

I've validated by generating a schema, added in this PR: AdobeDocs/commerce-webapi#555

@dshevtsov notes:

  • Added maintenance and refactoring changes

@jhadobe jhadobe requested a review from dshevtsov April 14, 2026 19:08
@jhadobe jhadobe self-assigned this Apr 14, 2026
@jhadobe jhadobe added the enhancement New feature or request label Apr 14, 2026
@jhadobe jhadobe requested a review from dshevtsov April 14, 2026 20:24
@jhadobe
Copy link
Copy Markdown
Collaborator Author

jhadobe commented Apr 14, 2026

re-requesting review from @dshevtsov, since after discussions with @keharper i added some additional changes to the transformer

…ard rails

- Extract _edit_schema and _convert_schema helpers in edit_paas/convert_paas
  to eliminate duplicated one-liners
- Convert all inline Ruby -e strings to heredocs for readability
- Add rescue JSON::ParserError with clean error messages across all functions
- Add file existence checks in edit_accs and convert_accs
- Add info key validation in edit_accs and _edit_schema; remove silent || {} fallbacks
- Fix unquoted $VERSION in convert_paas output paths
- Update edit_accs comment to reflect full scope of changes
Add missing title and version entries to the metadata list,
update description bullet to reference accs-intro.md, and
note the release version prompt in the usage NOTE.
@dshevtsov
Copy link
Copy Markdown

@jhadobe please check if this still works as expected after my changes.

@dshevtsov dshevtsov changed the title add intro reference add intro reference to the accs editor function Apr 15, 2026
@jhadobe
Copy link
Copy Markdown
Collaborator Author

jhadobe commented Apr 20, 2026

@jhadobe please check if this still works as expected after my changes.

it still works

@dshevtsov dshevtsov merged commit f887d1b into main Apr 20, 2026
2 checks passed
@dshevtsov dshevtsov deleted the jh_accs-intro branch April 20, 2026 17:57
@dshevtsov dshevtsov self-assigned this Apr 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants