itflow-org / itflow

Free and open-source web application for MSPs. Unifies IT documentation, ticketing, invoicing.
https://itflow.org
GNU General Public License v3.0
535 stars 139 forks source link

0.2.0.0 Make POST actions use functions that API V2 can use #918

Closed o-psi closed 5 months ago

o-psi commented 5 months ago

{WIP}

This should simplify the codebase with a branch in mind, and makes the API more maintainable.

wrongecho commented 5 months ago

Test these changes at: https://0200functionalizing918.pr-review.itflow.org
(automatic message)

johnnyq commented 5 months ago

Hey @o-psi there quite a bit to go through here. It will take me a few weeks to test and go through properly. We already have an API in place, although it could use additional end points as you stated. Also I don't want to change the POST code into functions. I want to keep the code style the way it is. So if you remove all those POST functions ill review the code again.

johnnyq commented 5 months ago

Also Can we put each small feature into its own pull request in the future as well. There no way my brain can not process this in a timely manor as i'm also working on core code duties. I'm not able to work efficiently or effectively this way.

For example Credits will have it own Pull Request. Update Ticket Statuses to use a function etc

o-psi commented 5 months ago

@johnnyq I think this may be our fork in the road.

For the version of this project I am maintaining, I will definitely use functions instead of hard coding it all.

I think it's a bit shortsighted, as a more modular codebase with less duplication will be easiest to maintain. In the end I was only using this project because it was what was best for our company, at this point I think what is best for my company is to hard fork.

Many thanks to all the work you have put into this package so far!

sonarcloud[bot] commented 5 months ago

Quality Gate Failed Quality Gate failed

Failed conditions
C Reliability Rating on New Code (required ≥ B)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint