[17.0][MIG] rma - Githubissues

AaronHForgeFlow commented 5 months ago

Migration of base rma module.

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.

@ForgeFlow

AaronHForgeFlow commented 5 months ago

DavidJForgeFlow commented 3 months ago

Hi Aaron, can you also include #498 ?

AaronHForgeFlow commented 3 months ago

@DavidJForgeFlow done. thanks

AaronHForgeFlow commented 3 months ago

AaronHForgeFlow commented 3 months ago

I think this is ready to be merged. If some issues arises we can take care of them in future PRs.

AaronHForgeFlow commented 2 months ago

I think it is time to merge this, further corrections will be done in future PRs.

ForgeFlow / stock-rma