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_account_nfe] #3273

Merged
merged 92 commits into from
Sep 6, 2024
Merged

[16.0][MIG][l10n_br_account_nfe] #3273

merged 92 commits into from
Sep 6, 2024

Conversation

rvalyi
Copy link
Member

@rvalyi rvalyi commented Aug 7, 2024

migraçao padrao em cima do PR da v15

depende de #2865

@rvalyi rvalyi force-pushed the 16.0-mig-l10n_br_account_nfe branch 7 times, most recently from e17265a to 6401b5d Compare August 8, 2024 05:47
@rvalyi
Copy link
Member Author

rvalyi commented Aug 8, 2024

botando como rascunho, pois ainda tem uns erros no test_nfe_generate_tags_cobr_dup_pag.py, qualquer ajuda para resolver eh bem vinda.

@rvalyi rvalyi marked this pull request as draft August 8, 2024 05:49
@rvalyi
Copy link
Member Author

rvalyi commented Aug 8, 2024

/ocabot migration l10n_br_account_nfe

@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
62 tasks
@rvalyi rvalyi changed the title [16.0][MIG][l10n_br_account_nfe] [16.0][MIG]l10n_br_account_nfe Aug 11, 2024
@rvalyi rvalyi changed the title [16.0][MIG]l10n_br_account_nfe [16.0][MIG] l10n_br_account_nfe Aug 11, 2024
@rvalyi rvalyi changed the title [16.0][MIG] l10n_br_account_nfe [16.0][MIG][l10n_br_account_nfe] Aug 11, 2024
@rvalyi
Copy link
Member Author

rvalyi commented Aug 28, 2024

@antoniospneto se conseguir ajudar nesse, seria bem vindo...

@antoniospneto antoniospneto mentioned this pull request Sep 4, 2024
Merged
@antoniospneto
Copy link
Contributor

@rvalyi

Fiz as correções e consegui rodar os testes do módulo localmente sem nenhum erro, é preciso destas PRs:

#3338
#3340

akretion#222

@rvalyi
Copy link
Member Author

rvalyi commented Sep 4, 2024 •
edited
Loading

Valeu! Aprovei os 2 e fiz o merge do terceiro. Depois que entrar irei dar um rebase. Importante tb se ligar que o port #3311 tb ainda não entrou na v16.

@rvalyi
Copy link
Member Author

rvalyi commented Sep 6, 2024

ja ja dou o rebase...

@rvalyi rvalyi force-pushed the 16.0-mig-l10n_br_account_nfe branch from 57c9cae to 946fa3c Compare September 6, 2024 18:21
weblate and others added 13 commits September 6, 2024 18:32
@weblate @rvalyi
Update translation files
895c0a4 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-brazil-14.0/l10n-brazil-14.0-l10n_br_account_nfe
Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-14-0/l10n-brazil-14-0-l10n_br_account_nfe/
@antoniospneto @rvalyi
[REF] l10n_br_account_nfe: rename nfce contingency function
8184263
@OCA-git-bot @rvalyi
[BOT] post-merge updates
1c8e306
@antoniospneto @rvalyi
[IMP] l10n_br_account_nfe: add unit test for nfe with IPI
b6d3847
@OCA-git-bot @rvalyi
[BOT] post-merge updates
acfa2f3
@CristianoMafraJunior @rvalyi
[IMP] l10n_br_account_nfe: print danfe
5275564
@antoniospneto @rvalyi
Translated using Weblate (Portuguese (Brazil))
551406d 
Currently translated at 80.0% (28 of 35 strings)

Translation: l10n-brazil-14.0/l10n-brazil-14.0-l10n_br_account_nfe
Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-14-0/l10n-brazil-14-0-l10n_br_account_nfe/pt_BR/
@rvalyi
[UPD] Update l10n_br_account_nfe.pot
457a380
@OCA-git-bot @rvalyi
[BOT] post-merge updates
3b7e675
@weblate @rvalyi
Update translation files
d650e47 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-brazil-14.0/l10n-brazil-14.0-l10n_br_account_nfe
Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-14-0/l10n-brazil-14-0-l10n_br_account_nfe/
@rvalyi
[REF] l10n_br_account_nfe: lint
acba583
@rvalyi
[MIG] l10n_br_account_nfe: Migration to 15.0
b9fd4a6
@rvalyi
[IMP] l10n_br_account_nfe: pre-commit stuff
d0eeee1
@rvalyi rvalyi force-pushed the 16.0-mig-l10n_br_account_nfe branch 2 times, most recently from 2a92679 to 4ec44fc Compare September 6, 2024 18:46
@rvalyi rvalyi force-pushed the 16.0-mig-l10n_br_account_nfe branch from 4ec44fc to 58752e2 Compare September 6, 2024 18:47
@rvalyi rvalyi marked this pull request as ready for review September 6, 2024 18:55
@rvalyi
Copy link
Member Author

rvalyi commented Sep 6, 2024

@renatonlima @marcelsavegnago @antoniospneto @mbcosta @felipemotter pronto para review, testes passando, valeu pro @antoniospneto ter ajudado a arrumar o PR.

@antoniospneto
Copy link
Contributor

/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-3273-by-antoniospneto-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit f4278cb into OCA:16.0 Sep 6, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 7828afb. 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 approved these changes

@antoniospneto antoniospneto antoniospneto approved these changes

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

Successfully merging this pull request may close these issues.