[15.0][MIG] rma_purchase_analytic

[JasminSForgeFlow]

No description provided.

[AaronHForgeFlow]

LGTM! Minor comments

[AaronHForgeFlow]

Thanks @JasminSForgeFlow some minor comments before the merge

[JasminSForgeFlow]

@AaronHForgeFlow it is now rebased, was issue in rebase

[codecov-commenter]

Codecov Report

Merging #447 (d1fc290) into 15.0 (4e7ebc7) will decrease coverage by 0.08%.
The diff coverage is 67.85%.

Additional details and impacted files

@@            Coverage Diff             @@
##             15.0     #447      +/-   ##
==========================================
- Coverage   80.68%   80.60%   -0.08%     
==========================================
  Files         144      149       +5     
  Lines        4830     4858      +28     
  Branches      754      756       +2     
==========================================
+ Hits         3897     3916      +19     
- Misses        720      728       +8     
- Partials      213      214       +1     

Files	Coverage Δ
rma_purchase_analytic/__init__.py	100.00% <100.00%> (ø)
rma_purchase_analytic/models/__init__.py	100.00% <100.00%> (ø)
rma_purchase_analytic/wizards/__init__.py	100.00% <100.00%> (ø)
...ma_purchase_analytic/models/purchase_order_line.py	62.50% <62.50%> (ø)
rma_purchase_analytic/wizards/rma_add_purchase.py	62.50% <62.50%> (ø)

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4e7ebc7...d1fc290. Read the comment docs.

[AaronHForgeFlow]

@JasminSForgeFlow , there are some merge commits in the commit history that should not be there. Also there are still conflicts in the PR so I can still not merge. Have you fetched the latest code in 15.0 before the rebase?

[JasminSForgeFlow]

@JasminSForgeFlow , there are some merge commits in the commit history that should not be there. Also there are still conflicts in the PR so I can still not merge. Have you fetched the latest code in 15.0 before the rebase?

I did rebase but maybe something went wrong. I have just followed the migration process and pushed it so it is up to date now

[AaronHForgeFlow]

@JasminSForgeFlow now the history looks good. Thanks.

[AaronHForgeFlow]

It works same as in previous versions.

I miss some features like filling in the analytic account in the RMA line when selecting a source PO, but that can be done in another PR.