Skip to content

[16.0] [ADD] shopinvader unit management (with members and request api)#1513

Open
paradoxxxzero wants to merge 26 commits intoshopinvader:16.0from
akretion:16.0-add-shopinvader_unit_management
Open

[16.0] [ADD] shopinvader unit management (with members and request api)#1513
paradoxxxzero wants to merge 26 commits intoshopinvader:16.0from
akretion:16.0-add-shopinvader_unit_management

Conversation

@paradoxxxzero
Copy link
Copy Markdown
Contributor

@paradoxxxzero paradoxxxzero commented Mar 7, 2024

Depends on #1508
Depends on #1490 and OCA/rest-framework#456 for shopinvader_fastapi_auth_partner_api_unit_member

@paradoxxxzero paradoxxxzero force-pushed the 16.0-add-shopinvader_unit_management branch from de25f29 to 1782719 Compare March 12, 2024 15:50
@sebastienbeau sebastienbeau added this to the 16.0 milestone Jun 3, 2024
@paradoxxxzero paradoxxxzero marked this pull request as ready for review June 4, 2024 13:51
@paradoxxxzero paradoxxxzero force-pushed the 16.0-add-shopinvader_unit_management branch 9 times, most recently from 1c0befa to aab2f26 Compare October 16, 2024 09:42
@paradoxxxzero paradoxxxzero force-pushed the 16.0-add-shopinvader_unit_management branch from 85be184 to 0bcede4 Compare October 18, 2024 19:51
@paradoxxxzero paradoxxxzero force-pushed the 16.0-add-shopinvader_unit_management branch from 0c9430f to 75d60a0 Compare April 3, 2025 09:34
@paradoxxxzero paradoxxxzero force-pushed the 16.0-add-shopinvader_unit_management branch from 4e7241a to 090f173 Compare April 10, 2025 14:52
@paradoxxxzero paradoxxxzero force-pushed the 16.0-add-shopinvader_unit_management branch from 090f173 to ca2c611 Compare June 2, 2025 11:14
Comment on lines +29 to +31
@cart_router.post("/{uuid}/request")
@cart_router.post("/current/request")
@cart_router.post("/request")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Create a new cart router, to avoid mounting it by error

@paradoxxxzero paradoxxxzero force-pushed the 16.0-add-shopinvader_unit_management branch from ca2c611 to 985a07e Compare September 1, 2025 10:16
@paradoxxxzero paradoxxxzero force-pushed the 16.0-add-shopinvader_unit_management branch from 985a07e to 732ac04 Compare November 18, 2025 09:17
@codecov
Copy link
Copy Markdown

codecov bot commented Nov 18, 2025

Codecov Report

❌ Patch coverage is 94.89489% with 51 lines in your changes missing coverage. Please review.
✅ Project coverage is 94.98%. Comparing base (6f56cba) to head (732ac04).
⚠️ Report is 30 commits behind head on 16.0.

Files with missing lines Patch % Lines
shopinvader_unit_management/models/res_partner.py 73.07% 12 Missing and 9 partials ⚠️
shopinvader_api_unit_request/models/sale_order.py 77.96% 5 Missing and 8 partials ⚠️
...auth_partner_api_unit_member/models/res_partner.py 86.48% 3 Missing and 2 partials ⚠️
...der_api_unit_request/routers/unit_request_lines.py 93.54% 3 Missing and 1 partial ⚠️
shopinvader_api_unit_request/schemas.py 90.00% 2 Missing and 2 partials ⚠️
shopinvader_api_unit_request/routers/cart.py 90.90% 1 Missing and 1 partial ⚠️
...request/tests/test_shopinvader_api_unit_request.py 99.21% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             16.0    #1513      +/-   ##
==========================================
- Coverage   95.00%   94.98%   -0.02%     
==========================================
  Files         525      555      +30     
  Lines       10876    11875     +999     
  Branches      553      617      +64     
==========================================
+ Hits        10333    11280     +947     
- Misses        386      414      +28     
- Partials      157      181      +24     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions
Copy link
Copy Markdown

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale label Mar 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants