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

[IMP] l10n_br_fiscal: adds icms st data #3422

Open
wants to merge 2 commits into
base: 14.0
Choose a base branch
from

Conversation

mileo
Copy link
Member

@mileo mileo commented Oct 7, 2024

This pull request includes several additions to the l10n_br_fiscal/data/operation_data.xml file to enhance the fiscal operation lines related to different types of sales and resales. The most important changes involve adding new records for various fiscal operations.

Additions to fiscal operation lines:

  • Added fo_venda_venda_st_substituto record for sales by substitute taxpayers. (l10n_br_fiscal/data/operation_data.xml)
  • Added fo_venda_revenda_st_substituto record for resale by substitute taxpayers. (l10n_br_fiscal/data/operation_data.xml)
  • Added fo_venda_revenda_st_substituido record for resale by substituted taxpayers. (l10n_br_fiscal/data/operation_data.xml)
  • Added fo_venda_revenda_st_substituido_sn record for resale by substituted taxpayers under the Simples Nacional regime. (l10n_br_fiscal/data/operation_data.xml)Adiciona dados de configuração para ST.

@OCA-git-bot
Copy link
Contributor

Hi @renatonlima,
some modules you are maintaining are being modified, check this out!

@antoniospneto
Copy link
Contributor

os testes falharam e não é dado demonstração certo?

@mileo mileo changed the title [IMP] l10n_br_fiscal: adds icms st demo data [IMP] l10n_br_fiscal: adds icms st data Oct 7, 2024
@antoniospneto antoniospneto marked this pull request as draft October 8, 2024 17:02
@Tiago370 Tiago370 force-pushed the 14.0-fiscal-st-data branch 4 times, most recently from 252192d to 0ce193b Compare November 1, 2024 18:46
@DiegoParadeda
Copy link
Contributor

DiegoParadeda commented Nov 1, 2024

image

acho que esse teste teria que testar != zero ao invés de == 4, pois o número de linhas de operação disponíveis pode ser alterado

@mileo mileo marked this pull request as ready for review November 5, 2024 12:50
Copy link
Contributor

@antoniospneto antoniospneto left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@marcelsavegnago marcelsavegnago left a comment

Choose a reason for hiding this comment

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

LGTM

@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). 🤖

@mileo
Copy link
Member Author

mileo commented Nov 5, 2024

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 14.0-ocabot-merge-pr-3422-by-mileo-bump-patch, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Nov 5, 2024
Signed-off-by mileo
@OCA-git-bot
Copy link
Contributor

@mileo your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-3422-by-mileo-bump-patch.

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.

@marcelsavegnago
Copy link
Member

@rvalyi seria bom um review de vcs já que são os autores.

@marcelsavegnago
Copy link
Member

marcelsavegnago commented Nov 5, 2024

@mileo se puder aguardar o review da akretion, seria bom.

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.

6 participants