Skip to content

Ajax callbacks on Next.Js EntityType settings form don't work. #845

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
Mschuddings opened this issue Mar 4, 2025 · 1 comment · May be fixed by #846
Open

Ajax callbacks on Next.Js EntityType settings form don't work. #845

Mschuddings opened this issue Mar 4, 2025 · 1 comment · May be fixed by #846
Labels
bug Something isn't working triage A new issue that needs triage

Comments

@Mschuddings
Copy link

Mschuddings commented Mar 4, 2025 •
edited
Loading

Package containing the bug

next (Drupal module)

Describe the bug

I've set-up the "module" following the documenation. When trying to get "Draft mode" to work. It seemed it was unable to select a "site".

Image

The part next to red arrow didn't show due to an JS error.

This problem also occurs in "On-demand revalidation" with the "Path" option.

Expected behavior

The Site selector should be visible so that I am able to select a site.

Steps to reproduce:

  1. go to: /admin/config/services/next/entity-types/node.article/edit?destination=/admin/config/services/next/entity-types
  2. Click vertical tab "Draft-mode" select "Site selector" (for example)
  3. See that the "Next.js sites" element doesn't show up.

Additional context

/
@Mschuddings Mschuddings added bug Something isn't working triage A new issue that needs triage labels Mar 4, 2025
@Mschuddings
Copy link
Author

Bug is resolved when:
'method' => 'replace',
is removed inside of "NextEntityTypeConfigForm" for both elements.

Mschuddings pushed a commit to Mschuddings/next-drupal that referenced this issue Mar 4, 2025
fix(next): Allow ajax callbakc inside of NextEntityTypeConfigForm to …
f1ae302 
…be executed.

Fixes chapter-three#845
@Mschuddings Mschuddings linked a pull request Mar 4, 2025 that will close this issue
Open
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working triage A new issue that needs triage
Projects
None yet
Milestone
No milestone
1 participant
@Mschuddings