Adding patch file.
martinpe → created an issue.
Patch added as a comment.
martinpe → changed the visibility of the branch 3516451-assertionerror-commerce-must to hidden.
martinpe → changed the visibility of the branch 3516451-assertionerror-commerce-must to hidden.
martinpe → created an issue.
I justed it and it's better to leave this code in.
Here is why:
- I started the project with
lando start, followed bylando npm run devto start the frontend - Then I used
./scripts/local/init-project.sh ddevto start the project with DDEV
and I had some problems such as
Port 80 is busy, using 33000 instead, see https://ddev.com/s/port-conflict
Port 443 is busy, using 33001 instead, see https://ddev.com/s/port-conflict
which changed the launch URL to https://starterkit.ddev.site:33001.
When I tried to access the site, it was not able to spin it up.
I think the main culprit is the traefik container (landoproxyhyperion5000gandalfedition) which needs to be fully stopped in order for DDEV to work. So there are some conflicts between DDEV and Lando containers.
Therefore I would leave that check untouched and maybe think of a different solution.
It's possible to configure DDEV to use different ports but just to run it all in parallel, that has to be researched.
You are right, that line is not needed.
Ready for review.
martinpe → created an issue.
martinpe → created an issue.
Patch added
martinpe → created an issue.
Adding updated patch which works with Drupal 10.3
martinpe → created an issue.
Updated patch for 2.0.0-beta6
martinpe → created an issue.
martinpe → created an issue.
Hello
Can somebody give me an update for this issue?