Skip to content

[MIG] l10n_br_purchase_blanket_order: Migration to 18.0#4763

Open
medicegabriel wants to merge 11 commits into
OCA:18.0from
Escodoo:18.0-mig-l10n_br_purchase_blanket_order
Open

[MIG] l10n_br_purchase_blanket_order: Migration to 18.0#4763
medicegabriel wants to merge 11 commits into
OCA:18.0from
Escodoo:18.0-mig-l10n_br_purchase_blanket_order

Conversation

@medicegabriel

Copy link
Copy Markdown

Migration from version 16 to 18

@OCA-git-bot OCA-git-bot added series:18.0 mod:l10n_br_purchase_blanket_order Module l10n_br_purchase_blanket_order labels Jul 24, 2026
@medicegabriel

Copy link
Copy Markdown
Author

PR is ready for review.

cc @marcelsavegnago @CristianoMafraJunior @kaynnan @WesleyOliveira98

@CristianoMafraJunior CristianoMafraJunior left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

[REVIEW FUNCIONAL]
Feito Criação de um Pedido Guarda-Chuva de Compra e a partir dele uma cotação
Image

@rvalyi

rvalyi commented Jul 25, 2026

Copy link
Copy Markdown
Member

/ocabot merge nobump

@CristianoMafraJunior

Copy link
Copy Markdown
Member

@medicegabriel Parabéns pela migração boa, alguns pontos mas não bloqueante:
1 - Esse commit 4112366 aqui poderia ta no commit de migração, só um store=True no campo como é algo da migração poderia tar junto mas nada impeditivo
2 - Colocar sempre versão da migração que ta fazendo ali ficaria: [18.0][MIG] l10n_br_purchase_blanket_order: Migration to 18.0

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 18.0-ocabot-merge-pr-4763-by-rvalyi-bump-nobump, awaiting test results.

@OCA-git-bot

Copy link
Copy Markdown
Contributor

@rvalyi your merge command was aborted due to failed check(s), which you can inspect on this commit of 18.0-ocabot-merge-pr-4763-by-rvalyi-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.

@rvalyi

rvalyi commented Jul 25, 2026

Copy link
Copy Markdown
Member

@medicegabriel cancelei o merge para dar uma chance de amendar os commits para atender o comentário do @CristianoMafraJunior . São detalhes, de uma forma geral ficou bom, valeu pela primeira contribuição!

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.

7 participants