End-to-end test that tests both the client (UI) and server

Wim Leers → credited lauriii → .

This would've prevented @lauriii from having to file 🐛 TypeError on a clean installation Closed: duplicate . He marked that as , so that helps justify this issue's priority 👍

📌 Add component instance edit form to contextual panel RTBC landed! This can now be worked on. 👍

MR I'm pushing up is built on top of 📌 Remove dependency on sdc_test Needs review as the changes there make it unnecessary to manually add components.

#5: 📌 End-to-end test that tests both the client (UI) and server Active is in 👍

AFAICT 🐛 Cypress teardown should check for dbUrl truthiness, not length Fixed and 🐛 hard codes ids, leading to multiple. Fixed were discovered while working on this?

Also: 📌 Cypress test infrastructure clean-up + baseline E2E test Active landed, so updating the issue summary :)

Found the root cause for the phpcs failures: 📌 CI: phpcs job failing due to upstream changes in Drupal core: unused variables in catch statements now disallowed Fixed .

bnjmnm → closed merge request !104

Looks great! Just one major question about appConfiguration (introduced in #3452582) vs drupalSettings, and I have a concrete proposal that AFAICT should work fine: https://git.drupalcode.org/project/experience_builder/-/merge_requests/1...

If @jessebaker could review it from the front-end perspective, then @bnjmnm can address our feedback and we can land this important milestone later today! 😊

Back to @bnjmnm - comments on MR

I think it just needs @Wim Leers signoff on the MR and we're good.

Just one remark about how both main.ts (to populate appConfiguration) and dummyPropsForm needed to interact with drupalSettings, but holding off merging this super important (because very stabilizing) MR on that makes no sense.

See https://git.drupalcode.org/project/experience_builder/-/merge_requests/1....

@hooroomoo is out today and is the owner for /submodules/xb_vite/. But the change being made to that is more Drupal-back-end-related than it is hot module reloading related, so I'm bypassing the need for their approval.

Now we're properly off to the races! 🐎 😄

Automatically closed - issue fixed for 2 weeks with no activity.