Conversation
BelleNottelling
left a comment
There was a problem hiding this comment.
Looks like a pretty decent start. I pushed to your branch to add formatting to the shell codeblocks and to include info on PHPStan.
Docker info looks to be complete and I'm happy with that and I'm indifferent about the remote installation page.
In the same subject of all of this, we also have a (very basic) Building FOSSBilling page which should probably be improved and then moved into the developer workflow.
It would also be nice to get those steps updated to include info on the new translation releases and how to set those up, but that's really an optional step & only matters if someone wants a dev instance that has all of the correct translations with their completion % available inside of FOSSBilling
|
Agreed on those points. I'll loop back to this after the installer and get those additions in. |
59276d4 to
0d00e2b
Compare
0d00e2b to
2562efd
Compare
Refer to: FOSSBilling/FOSSBilling#1583
This improves workflow documentation from the perspective of a developer new to FOSSBilling. Two options are documented:
I feel these updates accomplish the goal of clarifying new developer workflow, but I am open to making more improvements where needed.
The documentation does refer to a specific
installfolder handling with enablingdebugmode. That behavior does not exist yet, and is something I submit a pull request for. Refer to: FOSSBilling/FOSSBilling#1692