search

ForgeFlow / stock-rma

Stock RMA
GNU Affero General Public License v3.0
38 stars 41 forks source link

[17.0][MIG] rma_purchase #515

Open AaronHForgeFlow opened 5 months ago

AaronHForgeFlow commented 5 months ago

Supersedes https://github.com/ForgeFlow/stock-rma/pull/505

Respected original code & history

Reasons for supersede:

Merge 17.0 was broken after merging and reverting by mistake some pull requests: https://github.com/ForgeFlow/stock-rma/pull/483 & https://github.com/ForgeFlow/stock-rma/pull/500

History was incorrect showing merge commits within it. Merge conflicts when trying to manually merge other PRs on top on that one. Unable to see the changes in the migration commit, making hard for the reviewer to review the code

Sorry for the inconvenience.

https://github.com/ForgeFlow

AaronHForgeFlow commented 5 months ago

functional tests good so far. Need to investigate failing unit tests and also forward port this: #403

AaronHForgeFlow commented 5 months ago

Includes this now: https://github.com/ForgeFlow/stock-rma/pull/518/ (same as #403 )

AaronHForgeFlow commented 1 month ago

Pending to include https://github.com/ForgeFlow/stock-rma/pull/518 (when merged in 16)