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

[15.0][MIG] mrp_bom_line_sequence #975

Merged
merged 18 commits into from
Aug 10, 2023
Merged

[15.0][MIG] mrp_bom_line_sequence #975

merged 18 commits into from
Aug 10, 2023

Conversation

JuanyDForgeflow
Copy link

No description provided.

@JuanyDForgeflow JuanyDForgeflow force-pushed the 15.0-mig-mrp_bom_line_sequence branch from 6b95ee7 to 9e82a97 Compare February 25, 2023 02:15
@github-actions
Copy link

github-actions bot commented Jul 9, 2023

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 PR/Issue without recent activity, it'll be soon closed automatically. label Jul 9, 2023
AaronHForgeFlow and others added 18 commits July 20, 2023 09:56
@AaronHForgeFlow
[ADD]mrp_bom_line_sequence
204f47c
@OCA-git-bot @AaronHForgeFlow
[UPD] README.rst
5924896
@oca-travis @AaronHForgeFlow
[UPD] Update mrp_bom_line_sequence.pot
210af78
@OCA-git-bot @AaronHForgeFlow
[UPD] README.rst
5ff2e46
@MiquelRForgeFlow @AaronHForgeFlow
[MIG] mrp_bom_line_sequence: Migration to 12.0
dbf6013
@oca-travis @AaronHForgeFlow
[UPD] Update mrp_bom_line_sequence.pot
6e683a8
@badbole @AaronHForgeFlow
Added translation using Weblate (Croatian)
24b7a05
@badbole @AaronHForgeFlow
Translated using Weblate (Croatian)
309cff9 
Currently translated at 100.0% (8 of 8 strings)

Translation: manufacture-12.0/manufacture-12.0-mrp_bom_line_sequence
Translate-URL: https://translation.odoo-community.org/projects/manufacture-12-0/manufacture-12-0-mrp_bom_line_sequence/hr/
@ps-tubtim @AaronHForgeFlow
[IMP] mrp_bom_line_sequence: black, isort
c545787
@ps-tubtim @AaronHForgeFlow
[MIG] mrp_bom_line_sequence: Migration to 13.0
55da173
@oca-travis @AaronHForgeFlow
[UPD] Update mrp_bom_line_sequence.pot
79bbf60
@OCA-git-bot @AaronHForgeFlow
pre-commit update
5860564
@MiquelRForgeFlow @AaronHForgeFlow
[UPD] Eficent -> ForgeFlow
76f51e6
@OCA-git-bot @AaronHForgeFlow
[UPD] README.rst
9df6320
@OCA-git-bot @AaronHForgeFlow
[UPD] README.rst
acc46eb
@OCA-git-bot @AaronHForgeFlow
[UPD] README.rst
160ff98
@AaronHForgeFlow
[MIG] mrp_bom_line_sequence: Migration to 14.0
60b5b2b
@AaronHForgeFlow
[MIG] mrp_bom_line_sequence: Migration to 15.0
7c3f2eb
@AaronHForgeFlow AaronHForgeFlow force-pushed the 15.0-mig-mrp_bom_line_sequence branch from 9e82a97 to 7c3f2eb Compare July 20, 2023 07:56
@AaronHForgeFlow
Copy link
Contributor

AaronHForgeFlow commented Jul 20, 2023 •
edited
Loading

Edit: well, actually it works, because the sequence is not propagated to any other place in Odoo. Actually the description of the module is misleading because the sequence is not propagated anywhere, but standard Odoo will take the same order when creating an MO for the BOM. The refactor is not needed in this case.

AaronHForgeFlow
AaronHForgeFlow reviewed Jul 20, 2023
View reviewed changes
Copy link
Contributor

@AaronHForgeFlow AaronHForgeFlow left a comment

Choose a reason for hiding this comment

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

Fuctional tests ok!

@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jul 23, 2023
JasminSForgeFlow
JasminSForgeFlow approved these changes Aug 10, 2023
View reviewed changes
Copy link
Contributor

@JasminSForgeFlow JasminSForgeFlow left a comment

Choose a reason for hiding this comment

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

Functional Test

LoisRForgeFlow
LoisRForgeFlow approved these changes Aug 10, 2023
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.

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 15.0-ocabot-merge-pr-975-by-LoisRForgeFlow-bump-nobump, awaiting test results.

@LoisRForgeFlow LoisRForgeFlow changed the title [15.0][mig]-mrp_bom_line_sequence [15.0][MIG] mrp_bom_line_sequence Aug 10, 2023
@LoisRForgeFlow
Copy link
Contributor

/ocabot migration mrp_bom_line_sequence

@OCA-git-bot OCA-git-bot added this to the 15.0 milestone Aug 10, 2023
@OCA-git-bot OCA-git-bot mentioned this pull request Aug 10, 2023
Open
30 tasks
@OCA-git-bot OCA-git-bot merged commit 0a6a7a9 into OCA:15.0 Aug 10, 2023
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 8568368. 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

@JasminSForgeFlow JasminSForgeFlow JasminSForgeFlow approved these changes

@LoisRForgeFlow LoisRForgeFlow LoisRForgeFlow approved these changes

@AaronHForgeFlow AaronHForgeFlow AaronHForgeFlow approved these changes

Assignees
No one assigned
Labels
approved merged 🎉
Projects
None yet
Milestone
15.0
Development

Successfully merging this pull request may close these issues.
9 participants
@JuanyDForgeflow @AaronHForgeFlow @OCA-git-bot @LoisRForgeFlow @JasminSForgeFlow @oca-travis @MiquelRForgeFlow @badbole @ps-tubtim