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

[13.0][MIG] stock_inventory_chatter #815

Merged
merged 17 commits into from
Jan 8, 2020
Merged

[13.0][MIG] stock_inventory_chatter #815

merged 17 commits into from
Jan 8, 2020

Conversation

JoanSForgeFlow
Copy link
Contributor

@JoanSForgeFlow JoanSForgeFlow commented Dec 24, 2019 •
edited
Loading

Standard stock_inventory_chatter migration to v13.

cc ~ @ForgeFlow @LoisRForgeFlow

@rousseldenis rousseldenis added this to the 13.0 milestone Dec 24, 2019
@OCA-git-bot OCA-git-bot mentioned this pull request Dec 24, 2019
Closed
54 tasks
LoisRForgeFlow
LoisRForgeFlow reviewed Dec 24, 2019
View reviewed changes
Copy link
Contributor

@LoisRForgeFlow LoisRForgeFlow left a comment

Choose a reason for hiding this comment

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

Remember to separate the pre-commit first run and the actual migration changes in 2 commits (https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-13.0#technical-method-to-migrate-a-module-from-120-to-130-branch), this eases migration review.

stock_inventory_chatter/__manifest__.py Outdated Show resolved Hide resolved
stock_inventory_chatter/__manifest__.py Outdated Show resolved Hide resolved
LoisRForgeFlow
LoisRForgeFlow reviewed Dec 27, 2019
View reviewed changes
Copy link
Contributor

@LoisRForgeFlow LoisRForgeFlow left a comment

Choose a reason for hiding this comment

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

Some more comments after testing on runbot.

stock_inventory_chatter/models/stock.py Outdated Show resolved Hide resolved
stock_inventory_chatter/models/stock.py Outdated Show resolved Hide resolved
HviorForgeFlow
HviorForgeFlow suggested changes Dec 31, 2019
View reviewed changes
stock_inventory_chatter/data/stock_data.xml Outdated Show resolved Hide resolved
stock_inventory_chatter/models/stock.py Outdated Show resolved Hide resolved
MiquelRForgeFlow and others added 17 commits January 7, 2020 09:38
@MiquelRForgeFlow @JoanSForgeFlow
[9.0][ADD] stock_inventory_chatter
10f8497
@oca-transbot @JoanSForgeFlow
OCA Transbot updated translations from Transifex
1f805c9
@MiquelRForgeFlow @JoanSForgeFlow
[MIG] stock_inventory_chatter: Migration to 10.0
3dc281b
@oca-travis @JoanSForgeFlow
[UPD] Update stock_inventory_chatter.pot
602ced4
@xavierjimenez @JoanSForgeFlow
[MIG][11.0] stock_inventory_chatter
ca056c9
@oca-travis @JoanSForgeFlow
[UPD] Update stock_inventory_chatter.pot
6f47f76
@LoisRForgeFlow @JoanSForgeFlow
[12.0][MIG] stock_inventory_chatter
0936d57
@oca-travis @JoanSForgeFlow
[UPD] Update stock_inventory_chatter.pot
6c42a93
@oca-transbot @JoanSForgeFlow
Update translation files
f067420 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: stock-logistics-warehouse-12.0/stock-logistics-warehouse-12.0-stock_inventory_chatter
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-12-0/stock-logistics-warehouse-12-0-stock_inventory_chatter/
@JoanSForgeFlow
Translated using Weblate (German)
e311f68 
Currently translated at 85.7% (12 of 14 strings)

Translation: stock-logistics-warehouse-12.0/stock-logistics-warehouse-12.0-stock_inventory_chatter
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-12-0/stock-logistics-warehouse-12-0-stock_inventory_chatter/de/
@oca-travis @JoanSForgeFlow
[UPD] Update stock_inventory_chatter.pot
1541f9d
@oca-transbot @JoanSForgeFlow
Update translation files
b9d3cb2 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: stock-logistics-warehouse-12.0/stock-logistics-warehouse-12.0-stock_inventory_chatter
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-12-0/stock-logistics-warehouse-12-0-stock_inventory_chatter/
@OCA-git-bot @JoanSForgeFlow
[UPD] README.rst
183d764
@gurneyalex @JoanSForgeFlow
Translated using Weblate (French)
0a67f0a 
Currently translated at 100.0% (16 of 16 strings)

Translation: stock-logistics-warehouse-12.0/stock-logistics-warehouse-12.0-stock_inventory_chatter
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-12-0/stock-logistics-warehouse-12-0-stock_inventory_chatter/fr/
@liweijie0812 @JoanSForgeFlow
Translated using Weblate (Chinese (Simplified))
c526906 
Currently translated at 100.0% (16 of 16 strings)

Translation: stock-logistics-warehouse-12.0/stock-logistics-warehouse-12.0-stock_inventory_chatter
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-12-0/stock-logistics-warehouse-12-0-stock_inventory_chatter/zh_CN/
@JoanSForgeFlow
[IMP] stock_inventory_chatter: black, isort
f14bdd5
@JoanSForgeFlow
[MIG] stock_inventory_chatter: Migration to 13.0
90a165e
@JoanSForgeFlow JoanSForgeFlow force-pushed the 13.0-mig-stock_inventory_chatter branch from ced0824 to 90a165e Compare January 7, 2020 08:41
@JoanSForgeFlow JoanSForgeFlow reopened this Jan 7, 2020
AdriaGForgeFlow
AdriaGForgeFlow approved these changes Jan 8, 2020
View reviewed changes
Copy link

@AdriaGForgeFlow AdriaGForgeFlow left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@LoisRForgeFlow
Copy link
Contributor

/ocabot merge

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 13.0-ocabot-merge-pr-815-by-LoisRForgeFlow-bump-no, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Jan 8, 2020
@OCA-git-bot
Merge PR #815 into 13.0
7fdc0d4 
Signed-off-by LoisRForgeFlow
@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). 🤖

@OCA-git-bot
Copy link
Contributor

It looks like something changed on 13.0 in the meantime.
Let me try again (no action is required from you).
Prepared branch 13.0-ocabot-merge-pr-815-by-LoisRForgeFlow-bump-no, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Jan 8, 2020
@OCA-git-bot
Merge PR #815 into 13.0
1990e58 
Signed-off-by LoisRForgeFlow
@OCA-git-bot OCA-git-bot merged commit 90a165e into OCA:13.0 Jan 8, 2020
@OCA-git-bot
Copy link
Contributor

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

@LoisRForgeFlow LoisRForgeFlow deleted the 13.0-mig-stock_inventory_chatter branch January 8, 2020 13:23
@rousseldenis
Copy link
Sponsor Contributor

@LoisRForgeFlow Could you also check the box in #727 after merging? It is not (yet) automatic. Thanks

@LoisRForgeFlow
Copy link
Contributor

@rousseldenis Sure, will do next time. Thanks

CarlosRoca13 pushed a commit to Tecnativa/stock-logistics-warehouse that referenced this pull request Aug 16, 2021
@OCA-git-bot
Merge PR OCA#815 into 13.0
ec73cb1 
Signed-off-by LoisRForgeFlow
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LoisRForgeFlow LoisRForgeFlow LoisRForgeFlow left review comments

@HviorForgeFlow HviorForgeFlow HviorForgeFlow approved these changes

@AdriaGForgeFlow AdriaGForgeFlow AdriaGForgeFlow approved these changes

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

Successfully merging this pull request may close these issues.

None yet