Skip to content

Conversation

@rjaraspearhead
Copy link

Copy link
Contributor

@jbaudoux jbaudoux left a comment

Choose a reason for hiding this comment

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

LG, not tested

Copy link
Contributor

@jbaudoux jbaudoux left a comment

Choose a reason for hiding this comment

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

Out of curiosity, what is your usage of this module?

@rjaraspearhead rjaraspearhead force-pushed the 18.0-mig-stock_reserve branch 4 times, most recently from 2e458bb to 09a044f Compare March 12, 2025 13:43
@rjaraspearhead
Copy link
Author

@jbaudoux
The stock_reserve module is used to partially or fully reserve inventory, ensuring that the reserved quantity is not available for other operations.

Additionally, this module serves as a foundation for other extensions, such as stock_reserve_sale, which allows reservations to be created directly from the sales order without the need to confirm it.

@jbaudoux
Copy link
Contributor

@jbaudoux The stock_reserve module is used to partially or fully reserve inventory, ensuring that the reserved quantity is not available for other operations.

Additionally, this module serves as a foundation for other extensions, such as stock_reserve_sale, which allows reservations to be created directly from the sales order without the need to confirm it.

I was wondering if you were doing reservations outside sales.

For sales reservation, there is https://github.com/OCA/sale-workflow/pulls?q=is%3Apr+is%3Aopen+prebook since 14.0 which is better in my opinion. PRs are being moved to https://github.com/OCA/sale-prebook

@jbaudoux
Copy link
Contributor

/ocabot migration stock_reserve

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone May 20, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request May 20, 2025
8 tasks
@github-actions
Copy link

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 Sep 21, 2025
@ivs-cetmix
Copy link
Member

Hi @rjaraspearhead , thank you for your contribution!
Could you please rebase the PR?

@ivs-cetmix
Copy link
Member

/ocabot rebase

@OCA-git-bot
Copy link
Contributor

Sorry @ivs-cetmix you are not allowed to rebase.

To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons.

If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the maintainers key of its manifest.

@github-actions github-actions bot removed the stale label Oct 12, 2025
Guewen Baconnier and others added 9 commits October 12, 2025 17:37
…will serve as a basis for the sale pre-book
don't use half the quantity so that the reserved qty is different from
available qty
For weird reasons, force-assigning a move does not associate quants.
action_assign'ing the picking does, and is the same procedure that you
can do via the interface.
[UPG] Upgraded to version 8, fixed references to new 8.0  views and  moved fields that were in  product.product to product.template

[fix] remove duplicate view and correct view name

[UPG][FIX] added outgoing field XML, that was in the base stock field in 7.0.

[UPG] stock available immediately, corrected the calculation method of
immediately_usable_qty to take in accountthe sign change in outgoing_qty
(from negative to positive) in version 8.

[FLAKE8]

[FIX] renaming of a class, comment removing, useless code.

[UPD] move out from unported to 8 for update

[fix] remove duplicate view and correct view name

[UPG][FIX] added outgoing field XML, that was in the base stock field in 7.0.

[FIX] renaming of a class, comment removing, useless code.

[fix] remove duplicate view and correct view name

[UPG][FIX] added outgoing field XML, that was in the base stock field in 7.0.

[UPD] move out from unported to 8 for update

[fix] remove duplicate view and correct view name

[UPG][FIX] added outgoing field XML, that was in the base stock field in 7.0.

[FIX] renaming of a class, comment removing, useless code.

[UPD] move out from unported to 8 for update

[fix] remove duplicate view and correct view name

[UPG][FIX] added outgoing field XML, that was in the base stock field in 7.0.

[FIX] renaming of a class, comment removing, useless code.

[UPD] move out from unported to 8 for update

[fix] remove duplicate view and correct view name

[UPG][FIX] added outgoing field XML, that was in the base stock field in 7.0.

[FIX] renaming of a class, comment removing, useless code.

[fix] remove duplicate view and correct view name

[UPG][FIX] added outgoing field XML, that was in the base stock field in 7.0.

[UPD] move out from unported to 8 for update

[fix] remove duplicate view and correct view name

[UPG][FIX] added outgoing field XML, that was in the base stock field in 7.0.

[FIX] renaming of a class, comment removing, useless code.
mymage and others added 20 commits October 12, 2025 17:37
Currently translated at 16.1% (27 of 167 strings)

Translation: stock-logistics-warehouse-15.0/stock-logistics-warehouse-15.0-stock_reserve
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-15-0/stock-logistics-warehouse-15-0-stock_reserve/it/
Currently translated at 38.9% (65 of 167 strings)

Translation: stock-logistics-warehouse-15.0/stock-logistics-warehouse-15.0-stock_reserve
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-15-0/stock-logistics-warehouse-15-0-stock_reserve/sl/
Currently translated at 17.4% (27 of 155 strings)

Translation: stock-logistics-warehouse-16.0/stock-logistics-warehouse-16.0-stock_reserve
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-16-0/stock-logistics-warehouse-16-0-stock_reserve/it/
…tion to avoid multi-company incompatibilities

TT44108
Currently translated at 17.6% (27 of 153 strings)

Translation: stock-logistics-warehouse-16.0/stock-logistics-warehouse-16.0-stock_reserve
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-16-0/stock-logistics-warehouse-16-0-stock_reserve/it/
Currently translated at 100.0% (153 of 153 strings)

Translation: stock-logistics-warehouse-16.0/stock-logistics-warehouse-16.0-stock_reserve
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-16-0/stock-logistics-warehouse-16-0-stock_reserve/es/
Currently translated at 17.3% (27 of 156 strings)

Translation: stock-logistics-warehouse-16.0/stock-logistics-warehouse-16.0-stock_reserve
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-16-0/stock-logistics-warehouse-16-0-stock_reserve/it/
Currently translated at 100.0% (156 of 156 strings)

Translation: stock-logistics-warehouse-16.0/stock-logistics-warehouse-16.0-stock_reserve
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-16-0/stock-logistics-warehouse-16-0-stock_reserve/es/
Currently translated at 100.0% (156 of 156 strings)

Translation: stock-logistics-warehouse-16.0/stock-logistics-warehouse-16.0-stock_reserve
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-16-0/stock-logistics-warehouse-16-0-stock_reserve/it/
Currently translated at 100.0% (133 of 133 strings)

Translation: stock-logistics-warehouse-17.0/stock-logistics-warehouse-17.0-stock_reserve
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-17-0/stock-logistics-warehouse-17-0-stock_reserve/it/
Currently translated at 100.0% (133 of 133 strings)

Translation: stock-logistics-warehouse-17.0/stock-logistics-warehouse-17.0-stock_reserve
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-17-0/stock-logistics-warehouse-17-0-stock_reserve/es/
@rjaraspearhead
Copy link
Author

Hi @rjaraspearhead , thank you for your contribution! Could you please rebase the PR?

Done

@zalei-odoo
Copy link

I checked the functionality of the module.
The reservation of the product is expected if the product is less than the available quantity. It's all good.
In my opinion, the wrong behavior concerns a situation when a quantity of goods is reserved more than is available.
In this case, the Stork Reservations smart button displays a quantity of 0 instead of the specified quantity, although in the Forecasted smart button the quantity is minus.,
The action on the button also filters the result and does not show the reserve, because the default filter is set to filter the Partially Available status.
0+

1+ 2+

@ivs-cetmix
Copy link
Member

@zalei-odoo thank you for your feedback! Could you please approve the PR or request changes if needed?

@zalei-odoo
Copy link

I don't have enough rights.
Can I get access rights or ask someone to approve the PR

curl -s -H "Authorization: token <MY_TOKEN>" \
     https://api.github.com/repos/OCA/stock-logistics-reservation \
| jq '.permissions'
{
  "admin": false,
  "maintain": false,
  "push": false,
  "triage": false,
  "pull": true
}

@ivs-cetmix
Copy link
Member

@zalei-odoo looks weird, don't you have this menu?

image

@zalei-odoo
Copy link

zalei-odoo commented Oct 22, 2025

Thank you, Ivan, I haven't done this before, I really have it. I'll do it now)

Copy link

@zalei-odoo zalei-odoo left a comment

Choose a reason for hiding this comment

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

The module's functionality is functional enough for typical applications when working with products and warehouses. I checked the reservation (when the quantity for reservation is less than the available quantity) and the cron task "Release the stock reservation having a passed validity date".

rousseldenis pushed a commit to acsone/stock-logistics-reservation that referenced this pull request Nov 8, 2025
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.