freckle / guides

Freckle guides and best practices
MIT License
53 stars 7 forks source link

Add `PATCH` semantics #67

Closed mjgpy3 closed 3 years ago

mjgpy3 commented 3 years ago

Having recently converted our remaining PUT routes to PATCH I thought it would be nice to unify and formalize some of the differing practices in the API.

I believe that some of these formalizations could benefit our backend testing and frontend implementations.

If PATCH returns the same value as GET, and is the same route, then frontend

These formalizations are nice for the backend because

Note that our current PATCH endpoints do not conform to these guidelines, in many cases. If this guideline is accepted I will file debt for the places that don't conform for the backend guild to prioritize.

mjgpy3 commented 3 years ago

@joris974 I think and hope so!