Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[16.0][MIG] stock_inventory_discrepancy: Migration to version 16.0 #1841

Merged
merged 17 commits into from
Sep 27, 2023
Merged

[16.0][MIG] stock_inventory_discrepancy: Migration to version 16.0 #1841

merged 17 commits into from
Sep 27, 2023

Conversation

carolinafernandez-tecnativa
Copy link
Contributor

Standard migration to v16

@Tecnativa

TT44362

@pedrobaeza

@pedrobaeza
Copy link
Member

/ocabot migration stock_inventory_discrepancy

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Sep 13, 2023
@OCA-git-bot
Copy link
Contributor

The migration issue (#1494) has not been updated to reference the current pull request because a previous pull request (#1668) is not closed.
Perhaps you should check that there is no duplicate work.
CC @Urvisha-OSI

@pedrobaeza
Copy link
Member

Remember to squash administrative commits.

@pedrobaeza pedrobaeza mentioned this pull request Sep 13, 2023
Closed
@pedrobaeza
Copy link
Member

/ocabot migration stock_inventory_discrepancy

@OCA-git-bot OCA-git-bot mentioned this pull request Sep 13, 2023
Open
69 tasks
@carolinafernandez-tecnativa carolinafernandez-tecnativa force-pushed the 16.0-mig-stock_inventory_discrepancy branch 2 times, most recently from 92e5ccd to 0c8585e Compare September 14, 2023 16:05
@carolinafernandez-tecnativa
Copy link
Contributor Author

Remember to squash administrative commits.

@pedrobaeza done!

sergio-teruel
sergio-teruel approved these changes Sep 15, 2023
View reviewed changes
Copy link
Contributor

@sergio-teruel sergio-teruel left a comment

Choose a reason for hiding this comment

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

Thanks... Only minor comment

stock_inventory_discrepancy/tests/test_inventory_discrepancy.py Outdated Show resolved Hide resolved
@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

LoisRForgeFlow and others added 13 commits September 18, 2023 06:36
@LoisRForgeFlow @carolinafernandez-tecnativa
[9.0][ADD] stock_inventory_discrepancy (OCA#245)
7e6f37f 
[ADD] stock_inventory_discrepancy
@LoisRForgeFlow @carolinafernandez-tecnativa
[10.0][MIG] stock_inventory_discrepancy
cb76aaa
@carolinafernandez-tecnativa
[MIG] stock_inventory_discrepancy: Migration to 11.0
51630e4
@rousseldenis @carolinafernandez-tecnativa
[10.0][FIX] stock_inventory_discrepancy: Launch wizard as current user
b6d7d6c
@carolinafernandez-tecnativa
[12.0][MIG] stock_inventory_discrepancy: Migration to 12.0
cad37c3 
Added translation using Weblate (Chinese (Simplified))
@HviorForgeFlow @carolinafernandez-tecnativa
[IMP] stock_inventory_discrepancy: black, isort
a9610cb
@HviorForgeFlow @carolinafernandez-tecnativa
[MIG] stock_inventory_discrepancy: Migration to 13.0
201c85f
@MiquelRForgeFlow @carolinafernandez-tecnativa
[UPD] Eficent -> ForgeFlow
c035a58
@LoisRForgeFlow @carolinafernandez-tecnativa
[IMP] stock_inventory_discrepancy: centralize logic for over discrepa…
6088ede 
…ncy assessment and ease extension.
@MateuGForgeFlow @carolinafernandez-tecnativa
[14.0][MIG] stock_inventory_discrepancy: black, isort, prettier
3f51453
@MateuGForgeFlow @carolinafernandez-tecnativa
[14.0][MIG] stock_inventory_discrepancy: Migration to 14.0
0322e76
@dnplkndll @carolinafernandez-tecnativa
[imp] discrpency by product avg
e2daa7e
@dreispt @carolinafernandez-tecnativa
[REF] stock_inventory_discrepancy: fix eslint no-param-reassign
e0a5b46
Ernesto Tejeda and others added 2 commits September 18, 2023 06:36
@carolinafernandez-tecnativa
[MIG] stock_inventory_discrepancy: Migration to 15.0
60cbc06 
[UPD] Update stock_inventory_discrepancy.pot

[UPD] README.rst

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: stock-logistics-warehouse-15.0/stock-logistics-warehouse-15.0-stock_inventory_discrepancy
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-15-0/stock-logistics-warehouse-15-0-stock_inventory_discrepancy/
@pedrobaeza @carolinafernandez-tecnativa
[FIX] stock_inventory_discrepancy: Ensure active_ids value
590763d 
Steps to reproduce:

- Enter a product.
- Navigate to stock on hand.
- Adjust quantity and rebase discrepancy limit.
- Accept the discrepancy popup.

You get an error as the ID to be tried to adjust is the product one,
not the quant one.

We need to overwrite active_ids context with the quant IDs to be
handled on the wizard popup.

TT43947

stock_inventory_discrepancy 15.0.1.0.1

Translated using Weblate (Spanish)

Currently translated at 100.0% (31 of 31 strings)

Translation: stock-logistics-warehouse-15.0/stock-logistics-warehouse-15.0-stock_inventory_discrepancy
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-15-0/stock-logistics-warehouse-15-0-stock_inventory_discrepancy/es/

[UPD] README.rst
@pedrobaeza
Copy link
Member

The pre-commit stuff now contains the changes prior to this last one. I think you have squashed incorrectly.

@pedrobaeza pedrobaeza force-pushed the 16.0-mig-stock_inventory_discrepancy branch from a7c3c8d to 8a5cb3f Compare September 18, 2023 10:59
@carolinafernandez-tecnativa
Copy link
Contributor Author

ping @pedrobaeza

@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-1841-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 5f335b8 into OCA:16.0 Sep 27, 2023
6 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 2cb4cce. Thanks a lot for contributing to OCA. ❤️

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

@AungKoKoLin1997 AungKoKoLin1997 AungKoKoLin1997 left review comments

@rousseldenis rousseldenis rousseldenis left review comments

@pedrobaeza pedrobaeza pedrobaeza approved these changes

@sergio-teruel sergio-teruel sergio-teruel approved these changes

Assignees
No one assigned
Labels
approved merged 🎉 ready to merge
Projects
None yet
Milestone
16.0
Development

Successfully merging this pull request may close these issues.