search

ForgeFlow / stock-rma

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

[17.0] [MIG] rma_account #513

Closed AaronHForgeFlow closed 1 month ago

AaronHForgeFlow commented 5 months ago

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

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.

@ForgeFlow

AaronHForgeFlow commented 5 months ago

There are some fields in in the refund wizard that should not show to the user (technical fields that are empty)

image

AaronHForgeFlow commented 5 months ago

Already includes https://github.com/ForgeFlow/stock-rma/pull/519

AaronHForgeFlow commented 1 month ago

Pending to add https://github.com/ForgeFlow/stock-rma/pull/537

AaronHForgeFlow commented 1 month ago

Pending to add #538

AaronHForgeFlow commented 1 month ago

All FW port completed and working. Ready to merge.