You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As of #279 we're deprecating "open" and "close" on Adjustment in general. Shall we also change the /open_adjustments and /close_adjustments routes to /finalize_adjustments and /unfinalize_adjustments?
The text was updated successfully, but these errors were encountered:
I'm usually a fan of nesting resources where things make sense.
Rather then close_adjustments and unfinalize_adjustments I'd personally prefer /orders/:id/adjustments/finalize and /orders/:id/adjustments/unfinalize.
Also are we tied to finalize/unfinalize? I've decided I'm partial to "settle"/"unsettle", but mostly because I don't like the concept of "finalizing" something that is finalized.
See discussion here: #321 (comment)
As of #279 we're deprecating "open" and "close" on Adjustment in general. Shall we also change the /open_adjustments and /close_adjustments routes to /finalize_adjustments and /unfinalize_adjustments?
The text was updated successfully, but these errors were encountered: