[16][ADD] rma_category

[florian-dacosta]

Add the possibility to add one or more categories on RMA

[codecov-commenter]

Codecov Report

Merging #458 (8bcda1c) into 16.0 (282987b) will increase coverage by 0.05%.
The diff coverage is 100.00%.

Additional details and impacted files

@@            Coverage Diff             @@
##             16.0     #458      +/-   ##
==========================================
+ Coverage   80.98%   81.04%   +0.05%     
==========================================
  Files         124      128       +4     
  Lines        4807     4822      +15     
  Branches      779      779              
==========================================
+ Hits         3893     3908      +15     
  Misses        701      701              
  Partials      213      213              

Files	Coverage Δ
rma_category/__init__.py	100.00% <100.00%> (ø)
rma_category/models/__init__.py	100.00% <100.00%> (ø)
rma_category/models/rma_category.py	100.00% <100.00%> (ø)
rma_category/models/rma_order_line.py	100.00% <100.00%> (ø)

---

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 282987b...8bcda1c. Read the comment docs.

[AaronHForgeFlow]

LGTM, I would appreciate a test, but as long as there is no logic we can skip that.

[florian-dacosta]

LGTM, I would appreciate a test, but as long as there is no logic we can skip that.

Indeed, there are not logic at all, just a way to classify the RMA with a M2O. I would not know which test I could do for now !

[AaronHForgeFlow]

LGTM including the fix in RMA view. Can you please put the RMA change in a separate commit? Also, the readme won't be created automatically by the content in the readme folder, so I am afraid you have to do it :'(

[florian-dacosta]

LGTM including the fix in RMA view. Can you please put the RMA change in a separate commit? Also, the readme won't be created automatically by the content in the readme folder, so I am afraid you have to do it :'(

I did a separate commit.
About the README, I made the change already 2 days ago.

[AaronHForgeFlow]

Thank you @florian-dacosta

[florian-dacosta]

Well, pre-commit fails suddenly, I have no idea why...

[AaronHForgeFlow]

I will check that, perhaps we have to update pre-commit version

[AaronHForgeFlow]

It seems related to change in python version in github actions:

OCA/oca-addons-repo-template#214

[AaronHForgeFlow]

@florian-dacosta if you rebase it should be green now.

[florian-dacosta]

Great, thanks !