refactor(admin): Improve the PHP Modules & Configuration chapter#13573
refactor(admin): Improve the PHP Modules & Configuration chapter#13573
Conversation
Signed-off-by: Josh Richards <josh.t.richards@gmail.com>
Signed-off-by: Josh Richards <josh.t.richards@gmail.com>
|
Aside: I tapped GitHub Copilot to suggest an initial improvement baseline for rewriting this section. I was able to turn around a nice prototype in seconds. I disagreed with some of the changes so it took a couple more hours to make things how I wanted (through good old-fashioned manual editing). Based on this experiment, Copilot (or other similar AI agents) can be a helpful tool for refactoring our documentation. |
Signed-off-by: Josh <josh.t.richards@gmail.com>
| +------------------+----------+-------------+------------------+-----------------------------------------------+ | ||
| | imagick | | | ✓ (optional) | Image previews | | ||
| +------------------+----------+-------------+------------------+-----------------------------------------------+ | ||
| | avconv/ffmpeg | | | ✓ (optional) | Video previews | |
There was a problem hiding this comment.
This is a bit misleading, as avconf/ffmpeg/libreoffice are no PHP modules.
There was a problem hiding this comment.
agreed, but that's somewhat of a problem already on this page as-is. ;-) I'm thinking of adding a "Preparing your Server" (or similar) chapter as part of a broader reorg of the entire Installation/Deployment section. Those might better belong somewhere like that.
Signed-off-by: Josh Richards <josh.t.richards@gmail.com>
@come-nc Good catch. Fixed. |
☑️ Resolves
The current chapter covering how to prepare PHP for a new Nextcloud Server installation is fairly comprehensive, but I felt could benefit from some polishing.
Improved by (among other things):
I think the chapter's clarity has improved. And the new headings also make it easier to link to specific scenarios.
P.S. While doing this I noted that there was initially a commit in #12871 that elaborated on
gmpfor WebAuthn but it got chopped during a merge ofmasterback into that PR. Was that intentional or an accident? Cc: @susnux & @rakekniven🖼️ Screenshots
New
Previously: