Reviewed it, all my feedback has been resolved and so was the remaining items from previous reviews.
This is a draft CR, the issue it describes was never merged into Drupal Core (as of today, 2026-01-13)
greggles → credited tim.plunkett → .
tim.plunkett → created an issue.
tim.plunkett → made their first commit to this issue’s fork.
From #9:
How will this work once we have multiple Content templates enabled?
Now we only allow to edit the "Default / Full" content template, but that will change in the future.
Did this ever get answered?
tim.plunkett → made their first commit to this issue’s fork.
tim.plunkett → made their first commit to this issue’s fork.
Discussed with Utkarsh, Naren, Ted, and Sergiu today.
For building *landing pages*, we agreed that Figma integration would be the top priority.
But for generating actual content (i.e. nodes), the `.doc` / `.md` and `.csv` approach sounds promising.
tim.plunkett → made their first commit to this issue’s fork.
tim.plunkett → made their first commit to this issue’s fork.
Merged. Please open a follow-up next time @heyyo! Thanks.
tim.plunkett → made their first commit to this issue’s fork.
tim.plunkett → made their first commit to this issue’s fork.
tim.plunkett → made their first commit to this issue’s fork.
tim.plunkett → made their first commit to this issue’s fork.
tim.plunkett → made their first commit to this issue’s fork.
tim.plunkett → made their first commit to this issue’s fork.
tim.plunkett → made their first commit to this issue’s fork.
This is failing on CI with:
1 test triggered 1 PHP warning:
1) /builds/project/canvas/modules/canvas_ai/src/CanvasAiPageBuilderHelper.php:144
Undefined array key "props"
Adding the *original* issue that introduced this.
Reviewed with @justafish at DC Vienna, I'm signing off on this as the AJAX subsystem maintainer.
I would like to opt-in this module: http://www.drupal.org/project/conductor
It has 15 open and 58 total issues, and all maintainers agree we are ready to try GitLab issues. We understand we cannot revert this change and there may be some unexpected issues as early adopters.
We don't have a d.o issue open yet, but we're tracking #5 more generically.
I merged the basic MR for now, as it can't hurt.
Thanks!
tim.plunkett → made their first commit to this issue’s fork.
tim.plunkett → made their first commit to this issue’s fork.
tim.plunkett → made their first commit to this issue’s fork.
tim.plunkett → made their first commit to this issue’s fork.
tim.plunkett → made their first commit to this issue’s fork.
Merged, @justafish might clean up, but this is good enough for now!
tim.plunkett → made their first commit to this issue’s fork.
tim.plunkett → made their first commit to this issue’s fork.
tim.plunkett → made their first commit to this issue’s fork.
tim.plunkett → made their first commit to this issue’s fork.
Agreed that Scenario 2 can be done in a follow-up issue.
tim.plunkett → made their first commit to this issue’s fork.
tim.plunkett → made their first commit to this issue’s fork.
tim.plunkett → made their first commit to this issue’s fork.
tim.plunkett → made their first commit to this issue’s fork.
tim.plunkett → made their first commit to this issue’s fork.
This looks good but there are minor merge conflicts after
📌
Canvas AI: #3533079 Sub Issue: Move layout model to currentValuesRef instead of separate layoutModelRef
Needs review
Please set back to RTBC after fixing
tim.plunkett → made their first commit to this issue’s fork.
tim.plunkett → made their first commit to this issue’s fork.
tim.plunkett → made their first commit to this issue’s fork.