Skip to content

Comments

feat: DA Request for Publish plugin and Publish Request Inbox app#13

Draft
sagarsane wants to merge 5 commits intomainfrom
da-request-publish-app-plugin
Draft

feat: DA Request for Publish plugin and Publish Request Inbox app#13
sagarsane wants to merge 5 commits intomainfrom
da-request-publish-app-plugin

Conversation

@sagarsane
Copy link
Collaborator

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 #

Test URLs:

@aem-code-sync
Copy link

aem-code-sync bot commented Feb 20, 2026

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

  • Re-run PSI checks
  • Re-sync branch
Commits

@sagarsane sagarsane marked this pull request as draft February 20, 2026 04:37
@sagarsane sagarsane requested a review from ravuthu February 20, 2026 04:37
@aem-code-sync
Copy link

aem-code-sync bot commented Feb 20, 2026

Page Scores Audits Google
📱 / PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
🖥️ / Timeout Exceeded PSI

@aem-code-sync aem-code-sync bot temporarily deployed to da-request-publish-app-plugin February 20, 2026 04:42 Inactive
@aem-code-sync aem-code-sync bot temporarily deployed to da-request-publish-app-plugin February 20, 2026 21:11 Inactive
sagarsane and others added 2 commits February 21, 2026 23:55
- Add toRecipientEmail/toRFC2822Address helpers so both Gmail and WSU
  providers correctly handle { email, name } recipient objects as well
  as plain email strings
- Fix CC deduplication in handleRequestPublish to work with either format
- Remove stale TODO comment (CC was already implemented in sendEmailWSU)
- Add getApproversAndCCForPath in api.js and use it in resendPublishRequest
  so the inbox app's resend flow includes CC recipients, matching the
  behaviour of the plugin's submit/resend flows

Co-authored-by: Cursor <cursoragent@cursor.com>
Move publish-workflow-requests.json from the repo root to
/.da/publish-workflow-requests.json so it is not visible to
content authors browsing the DA file tree.

Co-authored-by: Cursor <cursoragent@cursor.com>
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