Skip to content

Feat symfony ux integration#100

Merged
dominikletica merged 67 commits into
dev-latestfrom
feat-symfony-ux-integration
Jun 15, 2026
Merged

Feat symfony ux integration#100
dominikletica merged 67 commits into
dev-latestfrom
feat-symfony-ux-integration

Conversation

@dominikletica

@dominikletica dominikletica commented Jun 14, 2026

Copy link
Copy Markdown
Member

Summary

  • Added Symfony UX-based UI/UX foundations: notification center, UI alert dispatcher/inbox/push flow, live polling endpoints, reusable Twig components, filter/autocomplete/dialog/copy/disclosure/tabs controllers, Chart/Map wrappers, and package-aware live endpoint/cookie consent hooks.
  • Integrated Mercure tooling and health checks with polling fallback for UI alerts.
  • Reworked live operations to run in the background with runner alerts, on-demand ActionLog details, success auto-redirect/reload, and safer trigger-button states.
  • Added GDPR-oriented cookie consent foundation and optional native notification preference gating.
  • Removed obsolete preview-only consent stub before PR.

Testing

  • bin/phpunit: OK, 1180 tests / 8004 assertions
  • bin/jstest: not yet implemented
  • bin/lint: PASS
  • Other (if not covered above): targeted render checks

Documentation

  • Updated project readme (README.md): Not required for this foundation slice.
  • Updated feature drafts (dev/draft/*.md)
  • Updated class map (dev/CLASSMAP.md)
  • Updated worklog (dev/WORKLOG.md)
  • Updated dev/user manuals (dev/manual/*.md / docs/*.md): Dev manual snippets updated; user-facing docs not required yet.

Additional Checks

Linked Issues / Discussions

Review Notes

  • Mercure push is best-effort and health-gated; queued alerts keep polling fallback semantics.
  • Cookie consent currently registers only necessary core cookies; optional cookies must come from explicit providers/packages.
  • Larger follow-ups are intentionally deferred: LiveComponent table conversion, Dropzone/Cropper media flows, richer notification settings, captcha live endpoints, and later splitting of the large operation overlay controller.

@dominikletica dominikletica self-assigned this Jun 14, 2026
@dominikletica dominikletica added the enhancement New feature or request label Jun 14, 2026

@dominikletica dominikletica left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Checks OK, ready-for-review.

@dominikletica dominikletica marked this pull request as ready for review June 14, 2026 03:18

@dominikletica dominikletica left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Addressed review findings.
PHPUnit OK (1239 Tests, 8231 Assertions).
JSTest OK (33 Tests).
All lint checks passed.
Continue review.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f2c1a05c3f

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/Setup/SetupRuntimeCommandRunner.php Outdated
Comment thread src/Controller/CookieConsentController.php Outdated
Comment thread src/View/Alert/UiAlertTopicFactory.php Outdated
Comment thread src/Privacy/Cookie/CookieConsentManager.php Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2acab888f8

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/Privacy/Cookie/CookieConsentResponseSubscriber.php Outdated
Comment thread src/View/Alert/UiAlertTopicFactory.php Outdated
Comment thread src/Controller/LiveEndpointController.php

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 01822f4ccd

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/View/Alert/UiAlertTopicFactory.php Outdated
Comment thread src/Controller/UserController.php Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b2ff10516d

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/Privacy/Cookie/CookieConsentResponseSubscriber.php Outdated
Comment thread src/Controller/AdminPackageController.php Outdated
Comment thread templates/components/CookieConsent.html.twig

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9dce0f9b57

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/Privacy/Cookie/CoreCookieConsentProvider.php

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6e6e8dd00c

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread templates/components/AlertStack.html.twig

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 228f50cd32

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread assets/controllers/operation_overlay_controller.js

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 96ba7dee38

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread assets/controllers/ui_alert_stream_controller.js Outdated
Comment thread src/View/Twig/UiAlertTwigExtension.php Outdated
Comment thread src/Core/Package/PackageFileSyntaxValidator.php Outdated

@dominikletica dominikletica left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Addressed review findings.
PHPUnit OK (1274 tests, 8324 assertions).
JSTest OK (37 Tests).
All lint checks passed.
Ready to merge.

@dominikletica dominikletica merged commit d65c631 into dev-latest Jun 15, 2026
4 checks passed
@dominikletica dominikletica deleted the feat-symfony-ux-integration branch June 15, 2026 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant