Skip to content

Fix FormGent enquiry page compatibility#2871

Merged
Armanul46 merged 2 commits into
sovware:developmentfrom
devShamim:hotfix/formgent-enquiry
Jun 9, 2026
Merged

Fix FormGent enquiry page compatibility#2871
Armanul46 merged 2 commits into
sovware:developmentfrom
devShamim:hotfix/formgent-enquiry

Conversation

@devShamim

Copy link
Copy Markdown
Member

Summary

  • Pin React 18 and compatible WordPress package versions for the FormGent enquiry bundle.
  • Add pnpm overrides so transitive Gutenberg packages resolve to compatible React 18-era packages.
  • Avoid FormGent protected API usage by preparing response fields locally in Directorist.
  • Support both FormGent answer component exports and provide a basic fallback renderer.
  • Rebuild the FormGent integration JS/CSS assets.

Root Cause

The enquiry page was loading a Gutenberg/DataViews dependency graph that could resolve incompatible React/Gutenberg primitives, causing React element objects to be rendered as invalid children. The details endpoint also depended on a protected FormGent controller method, and the modal expected a FormGent answer component export that is not present in the installed FormGent build.

Validation

  • php -l includes/classes/class-formgent.php
  • pnpm exec prettier --check package.json
  • git diff --check on touched files
  • Verified the served formgent-integration.js asset no longer contains React 19, react.transitional.element, @wordpress/primitives@4.47, or webpackMissingModule markers.

@devShamim devShamim changed the title [codex] Fix FormGent enquiry page compatibility Fix FormGent enquiry page compatibility Jun 3, 2026
@devShamim devShamim requested a review from Armanul46 June 3, 2026 14:30
@devShamim devShamim marked this pull request as ready for review June 3, 2026 14:30
@Armanul46 Armanul46 added this to the v8.7.4 milestone Jun 8, 2026

@RabbiIslamRony RabbiIslamRony left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks Good

@Armanul46 Armanul46 merged commit aff1bef into sovware:development Jun 9, 2026
1 check 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