Skip to content

fix: remove legacy section customization fields - #35

Merged
Zoltan Hricz (hriczzoli) merged 1 commit into
mainfrom
embedded-web-remove-legacy-section-configuration-fields-dxa-3772
Jul 23, 2026
Merged

fix: remove legacy section customization fields#35
Zoltan Hricz (hriczzoli) merged 1 commit into
mainfrom
embedded-web-remove-legacy-section-configuration-fields-dxa-3772

Conversation

@hriczzoli

@hriczzoli Zoltan Hricz (hriczzoli) commented Jul 23, 2026

Copy link
Copy Markdown
Contributor
  • remove writingStylePrompt and contentPrompt fields from the interactionOptions handler

Copilot AI review requested due to automatic review settings July 23, 2026 13:26
@linear-code

linear-code Bot commented Jul 23, 2026

Copy link
Copy Markdown

DXA-3772

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Removes legacy “section customization” fields from the public interaction options payload types and demo payloads, aligning the examples with the current supported API surface.

Changes:

  • Removes contentPrompt / writingStylePrompt from InlineTemplateSectionInstructions and PersonalTemplateSectionFields TypeScript types.
  • Updates demo interaction options payloads (HTML + React) to no longer include the removed fields.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
src/types/payloads.ts Removes legacy section prompt fields from exported payload interfaces.
demo/typescript-demo.html Updates the TypeScript demo payload JSON to drop legacy fields.
demo/react-demo.tsx Updates the React demo payload object to drop legacy fields.
demo/index.html Updates the basic demo payload JSON to drop legacy fields.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/types/payloads.ts
@hriczzoli
Zoltan Hricz (hriczzoli) merged commit 4322c71 into main Jul 23, 2026
6 checks passed
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