📝 Update DDEV doc for add-on #2754
Conversation
CollierCZ
force-pushed
the
ddev-integration
branch
from
0a97d6d
to
f987c33
Compare
There was a problem hiding this comment.
Added a few comments.
Looping in @rfay for another pair of eyes :)
docs/src/development/local/ddev.md
Outdated
| --- | ||
|
|
||
| [DDEV](https://ddev.readthedocs.io/en/stable/) is an open-source tool for local PHP development environments. | ||
| [DDEV](https://ddev.readthedocs.io/en/stable/) is an open-source tool for local PHP and Node.js development environments. |
There was a problem hiding this comment.
Node.js apps is not supported yet.
There was a problem hiding this comment.
Is that just the add-on? The repository description included Node.js, which is why I added it: https://github.com/drud/ddev
There was a problem hiding this comment.
AFAIK it doesn't support Node.js webapps yet. Right @rfay ?
There was a problem hiding this comment.
Node.js is significantly supported in DDEV, including webapps and other uses, but there is no mapping from Platform's support to DDEV's support.
There was a problem hiding this comment.
I'm not sure how to present that exactly. Guess I can leave out Node.js, just seemed unfortunate not to mention the support.
CollierCZ
requested review from
lolautruche,
AnouckColson and
rfay
and removed request for
rfay
|
|
||
| If your app requires services to run, you have two options for developing locally: | ||
|
|
||
| - [Tethered local development](./tethered.md) involves running your app on a local web server |
There was a problem hiding this comment.
Wow, I had no idea. I certainly wouldn't recommend this to people, would suggest that this be offered as a second, distant alternative, rather than offering it first.
| @@ -17,6 +18,14 @@ If you're starting from scratch, first [create a project from a PHP template]({{ | |||
|
|
|||
| Make sure your computer meets the [system requirements for DDEV](https://ddev.readthedocs.io/en/stable/#system-requirements). | |||
|
|
|||
| For the integration to run smoothly, you also need the following tools: | |||
There was a problem hiding this comment.
We'll have to remember to remove this when we fix that dependency. Ah the joys of maintaining docs multiple places.
CollierCZ
force-pushed
the
ddev-integration
branch
from
15040fc
to
d1950f2
Compare
CollierCZ
force-pushed
the
ddev-integration
branch
from
d1950f2
to
ac54230
Compare
Why
Closes #2486
What's changed
Updated local dev overview with more of a separation between Docker-based tools and others.
Updated DDEV steps for add-on.
Added visual differentiation on another level of navigation.