Skip to content

fix(ew-actions): when editor.hide publish is set, also hide publish for canvas - #666

Open
shsteimer wants to merge 2 commits into
mainfrom
hidepub
Open

fix(ew-actions): when editor.hide publish is set, also hide publish for canvas#666
shsteimer wants to merge 2 commits into
mainfrom
hidepub

Conversation

@shsteimer

@shsteimer shsteimer commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Mirrors da-live's da-title.js filterActions(): fetches org/site DA config docs on hash change, ORs any editor.hidePublish rows, and drops publish from the Send button's menu items when the current document's fullpath starts with a configured prefix.

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):

Fix #664

https://feature-ew-actions-hide-publish--da-nx--adobe.aem.live/

should merge with adobe/da-live#1245

shsteimer and others added 2 commits August 13, 2026 09:07
Mirrors da-live's da-title.js filterActions(): fetches org/site DA
config docs on hash change, ORs any editor.hidePublish rows, and
drops publish from the Send button's menu items when the current
document's fullpath starts with a configured prefix.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
getFirstSheet grabs whichever sheet is positionally first, which
silently breaks if a config doc has multiple sheets and "data" isn't
first. Look up the "data" sheet by name and only fall back to
positional lookup if it doesn't exist.

Co-Authored-By: Claude <noreply@anthropic.com>
@aem-code-sync

aem-code-sync Bot commented Aug 13, 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

@shsteimer shsteimer changed the title Hidepub fix(ew-actions): when editor.hide publish is set, also hide publish for canvas Aug 13, 2026
@shsteimer
shsteimer marked this pull request as ready for review August 13, 2026 18:41
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.

[ew] support editor.hidePublish

2 participants