Skip to content

feat(dialog): add fieldType hidden field to container v2 dialog#1843

Open
rismehta wants to merge 4 commits intodevfrom
rismehta/feat-container-fieldtype
Open

feat(dialog): add fieldType hidden field to container v2 dialog#1843
rismehta wants to merge 4 commits intodevfrom
rismehta/feat-container-fieldtype

Conversation

@rismehta
Copy link
Copy Markdown
Collaborator

Summary

  • Adds a hidden fieldType select field with value form to the container v2 _cq_dialog, consistent with all other AF2 component dialogs (textinput, numberinput, dropdown, etc.)
  • This exposes the container's fieldType via the Content API's componentDefinitions endpoint, enabling pre-submission type validation of patches targeting form container properties

Test plan

  • Deploy to AEM and verify GET /adobe/pages/<pageId>/content-definition returns fieldType with valueType: string and options: ["form"] for the container component
  • Verify the hidden field does not appear in the Edit Dialog UI

🤖 Generated with Claude Code

@github-actions github-actions bot force-pushed the rismehta/feat-container-fieldtype branch from 59b9b51 to aeb409e Compare March 27, 2026 09:35
@adobe-bot
Copy link
Copy Markdown

Accessibility Violations Found

Id Impact
aria-required-attr critical
empty-heading minor
label-title-only serious
target-size serious

3 similar comments
@adobe-bot
Copy link
Copy Markdown

Accessibility Violations Found

Id Impact
aria-required-attr critical
empty-heading minor
label-title-only serious
target-size serious

@adobe-bot
Copy link
Copy Markdown

Accessibility Violations Found

Id Impact
aria-required-attr critical
empty-heading minor
label-title-only serious
target-size serious

@adobe-bot
Copy link
Copy Markdown

Accessibility Violations Found

Id Impact
aria-required-attr critical
empty-heading minor
label-title-only serious
target-size serious

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 27, 2026

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...e/components/internal/models/v1/form/TextImpl.java 0.00% 0 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

@adobe-bot
Copy link
Copy Markdown

Accessibility Violations Found

Id Impact
aria-required-attr critical
empty-heading minor
label-title-only serious
target-size serious

3 similar comments
@adobe-bot
Copy link
Copy Markdown

Accessibility Violations Found

Id Impact
aria-required-attr critical
empty-heading minor
label-title-only serious
target-size serious

@adobe-bot
Copy link
Copy Markdown

Accessibility Violations Found

Id Impact
aria-required-attr critical
empty-heading minor
label-title-only serious
target-size serious

@adobe-bot
Copy link
Copy Markdown

Accessibility Violations Found

Id Impact
aria-required-attr critical
empty-heading minor
label-title-only serious
target-size serious

rismehta and others added 4 commits March 30, 2026 11:46
Adds a hidden select field for fieldType with value 'form' to the
container v2 dialog, consistent with all other AF2 component dialogs.
This exposes the container's fieldType via the Content API's
componentDefinitions endpoint, enabling type validation of patches
targeting form container properties.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions github-actions bot force-pushed the rismehta/feat-container-fieldtype branch from 785b38c to 2d1aff2 Compare March 30, 2026 11:46
@adobe-bot
Copy link
Copy Markdown

Accessibility Violations Found

Id Impact
aria-required-attr critical
empty-heading minor
label-title-only serious
page-has-heading-one moderate
target-size serious

@adobe-bot
Copy link
Copy Markdown

Accessibility Violations Found

Id Impact
aria-required-attr critical
empty-heading minor
label-title-only serious
target-size serious

@adobe-bot
Copy link
Copy Markdown

Accessibility Violations Found

Id Impact
aria-required-attr critical
empty-heading minor
label-title-only serious
page-has-heading-one moderate
target-size serious

@adobe-bot
Copy link
Copy Markdown

Accessibility Violations Found

Id Impact
aria-required-attr critical
empty-heading minor
label-title-only serious
target-size serious

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