Skip to content

feat: add skills editor page with ew provider block#949

Open
anfibiacreativa wants to merge 2 commits into
mainfrom
ew-skills-block-class
Open

feat: add skills editor page with ew provider block#949
anfibiacreativa wants to merge 2 commits into
mainfrom
ew-skills-block-class

Conversation

@anfibiacreativa

Copy link
Copy Markdown
Member

Summary

  • Adds apps/skills.md as the /apps/skills page in da-live
  • Uses ew-skills block class so loadBlock resolves via providers.ew to ew-extensions (da-skills)

Why

  • The skills editor moved from da-nx to da-skills (ew-extensions)
  • da-nx [LOC] Find fragments in sheets #446 added providers.ew routing in loadBlock, but the consumer page needs the ew- prefix for the resolution to work
  • The old nx-skills-editor class pointed at a block that no longer exists in da-nx

What Changed

  • apps/skills.md (new file) -- page shell with <div class="ew-skills"></div> so loadBlock matches the ew provider and loads blocks/skills/skills.js from ew-extensions

Test Plan

  • Start da-live (:3000), da-nx (:6456 on ew-provider-support-v2), da-skills (:3001)
  • Navigate to http://localhost:3000/apps/skills?nx=local&nxver=2#/{org}/{site}
  • Verify the skills editor loads from :3001/blocks/skills/skills.js
  • Also update the content bus page at content.da.live/adobe/da-live/apps/skills to use ew-skills

Risks / Follow-ups

  • The content bus at content.da.live still serves the old nx-skills-editor class. Until that authored page is updated, the AEM CLI dev server will serve stale content from the remote mount, not this local file.
  • Depends on da-nx #446 for providers.ew support
  • Depends on da-skills ew-block-contract branch for blocks/skills/skills.js entry point

Adds /apps/skills as a da-live page that renders the ew-skills block.
loadBlock resolves the ew- prefix via providers.ew to load the skills
editor from ew-extensions (da-skills) instead of da-nx.

Co-authored-by: Cursor <cursoragent@cursor.com>
@anfibiacreativa anfibiacreativa requested a review from mhaack May 20, 2026 12:20
@aem-code-sync

aem-code-sync Bot commented May 20, 2026

Copy link
Copy Markdown

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch.
In case there are problems, just click the checkbox below to rerun the respective action.

  • Re-sync branch
Commits

loadBlock needs providers.ew to resolve ew-* blocks (like ew-skills)
to the ew-extensions AEM site. Without this, the provider lookup
returns undefined and the block fails to load.

Co-authored-by: Cursor <cursoragent@cursor.com>
@mhaack

mhaack commented May 21, 2026

Copy link
Copy Markdown
Contributor

@anfibiacreativa @hannessolo IHMO this can be closed, not needed any more.

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