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.

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

[16.0][MIG][l10n_br_sale] #2911

Merged
merged 943 commits into from
Aug 25, 2024
Merged

[16.0][MIG][l10n_br_sale] #2911

merged 943 commits into from
Aug 25, 2024

Conversation

rvalyi
Copy link
Member

@rvalyi rvalyi commented Feb 6, 2024

@rvalyi rvalyi marked this pull request as draft February 6, 2024 02:08
@rvalyi rvalyi force-pushed the 16.0-mig-l10n_br_sale branch 4 times, most recently from 59405a3 to a9cf044 Compare February 6, 2024 14:31
@renatonlima renatonlima added this to the 16.0 milestone Feb 6, 2024
@rvalyi rvalyi force-pushed the 16.0-mig-l10n_br_sale branch 5 times, most recently from d56f916 to ff09c70 Compare February 7, 2024 00:18
@rvalyi
Copy link
Member Author

rvalyi commented Feb 9, 2024

/ocabot migration l10n_br_sale

@OCA-git-bot OCA-git-bot mentioned this pull request Feb 9, 2024
58 tasks
@rvalyi rvalyi force-pushed the 16.0-mig-l10n_br_sale branch 2 times, most recently from 0789ccc to 82f1a19 Compare February 9, 2024 13:51
@rvalyi rvalyi force-pushed the 16.0-mig-l10n_br_sale branch 4 times, most recently from 7e1d268 to 997e090 Compare March 24, 2024 05:15
@rvalyi
Copy link
Member Author

rvalyi commented Mar 24, 2024

pelo menos desde que eu fiz depender do PR #2974 instala no Runboat. Agora é so ir acertando os testes...
A maioria que não passa é relacionado a necessidade de migrar o compute_amount no PR do l10n_br_account #2865

@rvalyi rvalyi force-pushed the 16.0-mig-l10n_br_sale branch 2 times, most recently from 5271472 to 1efc8b7 Compare August 8, 2024 06:39
@rvalyi
Copy link
Member Author

rvalyi commented Aug 8, 2024

@renatonlima @mbcosta @marcelsavegnago @antoniospneto @felipemotter ta quase passando todos testes ja. Apenas tem alguma treta com o valor do frete. Qualquer ajuda para arrumar isso esta bem vinda...

@rvalyi rvalyi force-pushed the 16.0-mig-l10n_br_sale branch 2 times, most recently from f64b486 to 18ae456 Compare August 23, 2024 04:43
@rvalyi rvalyi marked this pull request as ready for review August 23, 2024 04:52
mbcosta and others added 17 commits August 24, 2024 19:18
…l', necessary for the new version of pre-commit.
…ale lines when Sale Order has Fiscal Operation.
…and _get_invoiceable_lines method expects objects not a List
Signed-off-by: Luis Felipe Miléo <[email protected]>
@rvalyi rvalyi force-pushed the 16.0-mig-l10n_br_sale branch 2 times, most recently from 635dd42 to 9051978 Compare August 24, 2024 20:08
@rvalyi
Copy link
Member Author

rvalyi commented Aug 24, 2024

/ocabot merge nobump

@rvalyi
Copy link
Member Author

rvalyi commented Aug 25, 2024

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 16.0-ocabot-merge-pr-2911-by-rvalyi-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit c5b5d6d into OCA:16.0 Aug 25, 2024
6 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 321ec89. 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.