- Issue created by @effulgentsia
- 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
This is why Drupal's AJAX response system puts this information in the response body, not the response header.
I think it'd be good to take a good hard look at why exactly we're not using core's AJAX response system and
drupal_ajax
wrapper format.I suspect that reusing 99% of it and just introducing a new AJAX command (much like
class OpenModalDialogCommand extends OpenDialogCommand
) could be sufficient? - Merge request !401#3481343 do not provide asset data in response header → (Merged) created by bnjmnm
- 🇺🇸United States bnjmnm Ann Arbor, MI
Needs a @traviscarden signoff because of the change to openapi.yml
- 🇫🇮Finland lauriii Finland
Clicked the merge because it looks like there's plenty of approvals there. 🚀
- 🇺🇸United States q0rban Atlanta, GA
Is it possible that this didn't get merged in? I'm still seeing the large response headers on the 0.x branch, and it looks like MR 401 is still open.
- First commit to issue fork.
- 🇺🇸United States hooroomoo
Thanks q0rban, looks like the first merge failed because of e2e tests failing. Marking to needs work.
-
effulgentsia →
committed e97a668f on 0.x authored by
bnjmnm →
Issue #3481343 by bnjmnm, wim leers, effulgentsia, traviscarden:...
-
effulgentsia →
committed e97a668f on 0.x authored by
bnjmnm →
Automatically closed - issue fixed for 2 weeks with no activity.