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][l10n_br_stock_account] #3275

Merged
merged 511 commits into from
Aug 26, 2024
Merged

[16.0][MIG][l10n_br_stock_account] #3275

merged 511 commits into from
Aug 26, 2024

Conversation

rvalyi
Copy link
Member

@rvalyi rvalyi commented Aug 8, 2024

migraçao para a v16 em cima da migraçao v15

depende de #2865

instala mas ainda nao passa os testes...

@rvalyi
Copy link
Member Author

rvalyi commented Aug 8, 2024

/ocabot migration l10n_br_stock_account

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Aug 8, 2024
@OCA-git-bot OCA-git-bot mentioned this pull request Aug 8, 2024
Open
58 tasks
@rvalyi rvalyi force-pushed the 16.0-mig-l10n_br_stock_account branch from 42d57e9 to 3348f43 Compare August 8, 2024 20:14
@rvalyi rvalyi marked this pull request as draft August 8, 2024 22:36
@rvalyi rvalyi changed the title [16.0][MIG] l10n_br_stock_account [16.0][MIG][l10n_br_stock_account] Aug 11, 2024
@rvalyi rvalyi force-pushed the 16.0-mig-l10n_br_stock_account branch 3 times, most recently from 09fa67b to 05aa6c4 Compare August 23, 2024 06:13
@rvalyi rvalyi marked this pull request as ready for review August 23, 2024 06:21
@rvalyi
Copy link
Member Author

rvalyi commented Aug 23, 2024

@renatonlima @marcelsavegnago @antoniospneto @mbcosta a migraçao ficou pronta para review, passou tudo certinho.

antoniospneto
antoniospneto reviewed Aug 23, 2024
View reviewed changes
.github/workflows/test.yml
exit 1
# exit 1
Copy link
Contributor

Choose a reason for hiding this comment

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

Isso aqui é necessário? vi que foi feito na outra PR tbm

Copy link
Member

Choose a reason for hiding this comment

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

agora mesclando o l10n_br_account e fazendo um rebase ai acho que nao precisa mais

@antoniospneto
Copy link
Contributor

@renatonlima @marcelsavegnago @antoniospneto @mbcosta a migraçao ficou pronta para review, passou tudo certinho.

Na v15 foi desabilitado alguns testes, esses será que não dá pra habilitar pra ver se agora passa? ou fica para resolver posteriormente?

@rvalyi
Copy link
Member Author

rvalyi commented Aug 23, 2024

hum ok, vou olhar isso, valeu por notar.

@rvalyi rvalyi force-pushed the 16.0-mig-l10n_br_stock_account branch 3 times, most recently from 59040dd to c7a48a7 Compare August 25, 2024 05:22
@rvalyi
Copy link
Member Author

rvalyi commented Aug 26, 2024

@antoniospneto realmente eu tinha desabilitado alguns testes na época que ainda tinha bastante problemas com a migração do l10n_br_account. Mas agora botei os testes originais de volta e passou tudo. Tb precisei de incluir o commit de #3198 para funcionar.

inclui todos os últimos commits da 14.0 no l10n_br_stock_account tb.

Cc @renatonlima @marcelsavegnago @mbcosta

@mileo @rvalyi
[NEW] Stock Company Fields
23afbc3 
Signed-off-by: Luis Felipe Mileo <mileo@kmee.com.br>

Signed-off-by: Luis Felipe Mileo <mileo@kmee.com.br>
@mileo @rvalyi
[FIX][WIP] Compute taxes on picking
9984bab 
Signed-off-by: Luis Felipe Mileo <mileo@kmee.com.br>
@mileo @rvalyi
[IMP] Default operation main company demo
4bb61ae 
Signed-off-by: Luis Felipe Mileo <mileo@kmee.com.br>
@mileo @rvalyi
[NEW] Comment on stock.picking
3464c71 
Signed-off-by: Luis Felipe Mileo <mileo@kmee.com.br>
@mileo @rvalyi
[IMP] Propagate stock.rule fields to stock.move on pull rules
e6eabd8 
Signed-off-by: Luis Felipe Mileo <mileo@kmee.com.br>
@mileo @rvalyi
[IMP] Demo data and tests
aa4867c 
Signed-off-by: Luis Felipe Mileo <mileo@kmee.com.br>
@mileo @rvalyi
[FIX] pep8
4661773 
Signed-off-by: Luis Felipe Mileo <mileo@kmee.com.br>
@mileo @rvalyi
[REM] Duplicated demo data
3c7a8da 
Signed-off-by: Luis Felipe Mileo <mileo@kmee.com.br>
oca-ci and others added 17 commits August 26, 2024 15:12
@rvalyi
[UPD] Update l10n_br_stock_account.pot
f5a3c59
@OCA-git-bot @rvalyi
[BOT] post-merge updates
fbb5e9e
@weblate @rvalyi
Update translation files
944104a 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-brazil-14.0/l10n-brazil-14.0-l10n_br_stock_account
Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-14-0/l10n-brazil-14-0-l10n_br_stock_account/
@mbcosta @rvalyi
[REF] l10n_br_stock_account: Adapt code for the cases without Fiscal …
c6ce6aa 
…Operation, if a Purchase or Sale Order are defined without it the Picking should be the same.
@mbcosta @rvalyi
[REF] l10n_br_stock_account: Unnecessary call methods because the _on…
d292bda 
…change_product_id_fiscal already call it.
@OCA-git-bot @rvalyi
[BOT] post-merge updates
fb6487a
@mileo @rvalyi
[FIX] stock_account hasattr
5df6d9a 
Signed-off-by: Luis Felipe Miléo <mileo@kmee.com.br>
@marcelsavegnago @rvalyi
[FIX] l10n_br_stock_account: change res.company data to noupdate = 1
0fef42d
@DiegoParadeda @rvalyi
[IMP] set picking fop at creation from PO
7a8783f
@DiegoParadeda @rvalyi
[RMV] redundancy: 'fop' already comes from SO whether set or not
1ecb05a
@OCA-git-bot @rvalyi
[BOT] post-merge updates
5786b4c
@rvalyi
[UPD] Update l10n_br_stock_account.pot
f8d5310
@weblate @rvalyi
Update translation files
2ce590c 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-brazil-14.0/l10n-brazil-14.0-l10n_br_stock_account
Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-14-0/l10n-brazil-14-0-l10n_br_stock_account/
@rvalyi
[REF] l10n_br_stock_account: lint
a198f5b
@rvalyi
[MIG] l10n_br_stock_account: Migration to 15.0
e140a67
@rvalyi
[IMP] l10n_br_stock_account: pre-commit stuff
b1bed55
@rvalyi
[MIG] l10n_br_stock_account: Migration to 16.0
209d8be
@rvalyi rvalyi force-pushed the 16.0-mig-l10n_br_stock_account branch from c7a48a7 to 209d8be Compare August 26, 2024 15:13
@OCA OCA deleted a comment from OCA-git-bot Aug 26, 2024
@renatonlima
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-3275-by-renatonlima-bump-nobump, awaiting test results.

@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 OCA-git-bot merged commit 9bba617 into OCA:16.0 Aug 26, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@marcelsavegnago marcelsavegnago marcelsavegnago left review comments

@renatonlima renatonlima renatonlima approved these changes

@antoniospneto antoniospneto antoniospneto approved these changes

@douglascstd douglascstd Awaiting requested review from douglascstd

@vanderleiromera vanderleiromera Awaiting requested review from vanderleiromera

@WesleyOliveira98 WesleyOliveira98 Awaiting requested review from WesleyOliveira98

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.