Page MenuHomePhorge

Add small documentation details

Authored by valerio.bozzolan on Feb 9 2023, 17:19.
Referenced Files
Unknown Object (File)
Fri, Mar 10, 17:41
Unknown Object (File)
Thu, Mar 2, 03:51
Unknown Object (File)
Thu, Mar 2, 03:51
Unknown Object (File)
Thu, Mar 2, 03:50
Unknown Object (File)
Thu, Mar 2, 03:50
Unknown Object (File)
Thu, Mar 2, 03:49
Unknown Object (File)
Thu, Mar 2, 03:49
Unknown Object (File)
Wed, Mar 1, 12:27



Add some small Diviner documentation details like:

  • mention PHP 8.1 (thanks
  • mention the fact that you need multiple databases (it may be not obvious from the startup)
  • mention MariaDB and not just MySQL (thanks MariaDB community!)
  • mention "Administrator" and not just "User" for the "Instructions for installing, configuring, and using Phorge."

I was not bold enough to do more changes.

Test Plan

I tested locally and Diviner does not look bad.

Diff Detail

rP Phorge
Lint Not Applicable
Tests Not Applicable

Event Timeline

  • mention "Administrator" and not just "User" (not just end-users)
valerio.bozzolan retitled this revision from add more documentation details to Add small documentation details.Feb 9 2023, 17:24
valerio.bozzolan edited the summary of this revision. (Show Details)

This is just a small change but if you want a spoiler, here how it looks like:

Diviner proposed home.png (350×877 px, 43 KB)

Note the first section, that now it's not just for "Users" but it also mentions "Administrator".

(I know that an "administrator" is a kind of "user", but some administrators never read "user documentation", and that's why I think this small change could somehow better orient prominently).

avivey added inline comments.

Is there a way to phrase this it can't be understood as "multiple mysql servers"? This term-overloading always bugged me.

  • rephrase sentence with tip from avivey

I've done my best to rephrase that. Thanks @avivey

  • rephrase also the other sentence with tip from avivey
This revision is now accepted and ready to land.Feb 10 2023, 21:40

(If I remember correctly the rules we set for rP, you should be able to arc land all of these now)

This revision was automatically updated to reflect the committed changes.