Skip to content

[16.0] [ADD] shopinvader_api_warehouse#1610

Merged
shopinvader-git-bot merged 2 commits into
shopinvader:16.0from
akretion:16.0-add-shopinvader_api_warehouse
Jun 4, 2026
Merged

[16.0] [ADD] shopinvader_api_warehouse#1610
shopinvader-git-bot merged 2 commits into
shopinvader:16.0from
akretion:16.0-add-shopinvader_api_warehouse

Conversation

@paradoxxxzero

Copy link
Copy Markdown
Contributor

This Odoo module extends the Shopinvader API to add warehouse management for carts
(and more generally in sales).
It also saves the default warehouse per customer and exposes warehouses informations.

Warehouses are meant to be physical stores in this context.

@paradoxxxzero paradoxxxzero force-pushed the 16.0-add-shopinvader_api_warehouse branch 2 times, most recently from 519d5cb to 84cba6e Compare December 15, 2025 17:40
@codecov

codecov Bot commented Dec 15, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 96.65552% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 95.13%. Comparing base (fd34c42) to head (f2b237c).

Files with missing lines Patch % Lines
shopinvader_api_warehouse/schemas/warehouse.py 77.77% 4 Missing and 4 partials ⚠️
shopinvader_api_warehouse/routers/warehouse.py 93.54% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             16.0    #1610      +/-   ##
==========================================
+ Coverage   95.09%   95.13%   +0.04%     
==========================================
  Files         494      510      +16     
  Lines       10168    10467     +299     
  Branches      513      519       +6     
==========================================
+ Hits         9669     9958     +289     
- Misses        354      360       +6     
- Partials      145      149       +4     

☔ View full report in Codecov by Harness.
📢 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 May 17, 2026
@paradoxxxzero paradoxxxzero marked this pull request as ready for review May 18, 2026 16:07

@sebastienbeau sebastienbeau left a comment

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.

LGTM (already merged in 18)

@sebastienbeau

Copy link
Copy Markdown
Contributor

/ocabot merge nobump

@shopinvader-git-bot

Copy link
Copy Markdown

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-1610-by-sebastienbeau-bump-nobump, awaiting test results.

@shopinvader-git-bot

Copy link
Copy Markdown

@sebastienbeau your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-1610-by-sebastienbeau-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

@sebastienbeau

Copy link
Copy Markdown
Contributor

/ocabot merge nobump

@shopinvader-git-bot

Copy link
Copy Markdown

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-1610-by-sebastienbeau-bump-nobump, awaiting test results.

@shopinvader-git-bot

Copy link
Copy Markdown

@sebastienbeau your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-1610-by-sebastienbeau-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

@sebastienbeau sebastienbeau force-pushed the 16.0-add-shopinvader_api_warehouse branch from 73bc22a to 54eed2c Compare May 27, 2026 14:06
@sebastienbeau

Copy link
Copy Markdown
Contributor

/ocabot merge nobump

@shopinvader-git-bot

Copy link
Copy Markdown

This PR looks fantastic, let's merge it!
Prepared branch 16.0-ocabot-merge-pr-1610-by-sebastienbeau-bump-nobump, awaiting test results.

@shopinvader-git-bot

Copy link
Copy Markdown

@sebastienbeau your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-1610-by-sebastienbeau-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

@paradoxxxzero paradoxxxzero force-pushed the 16.0-add-shopinvader_api_warehouse branch from 54eed2c to f2b237c Compare June 4, 2026 10:05
@paradoxxxzero

Copy link
Copy Markdown
Contributor Author

/ocabot merge nobump

@shopinvader-git-bot

Copy link
Copy Markdown

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-1610-by-paradoxxxzero-bump-nobump, awaiting test results.

@shopinvader-git-bot shopinvader-git-bot merged commit 7d65442 into shopinvader:16.0 Jun 4, 2026
5 checks passed
@shopinvader-git-bot

Copy link
Copy Markdown

Congratulations, your PR was merged at 7b09ee9. Thanks a lot for contributing to shopinvader. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants