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

[14.0][IMP] envia instrução de protesto na primeira instrução do Itau. #3120

Merged
merged 1 commit into from
Jun 19, 2024
Merged

[14.0][IMP] envia instrução de protesto na primeira instrução do Itau. #3120

merged 1 commit into from
Jun 19, 2024

Conversation

mileo
Copy link
Member

@mileo mileo commented Jun 11, 2024 •
edited
Loading

lib/brcobranca/remessa/cnab400/itau.rb


        def prazo_instrucao(pagamento)
          return '03' unless pagamento.cod_primeira_instrucao == '09'

          pagamento.dias_protesto.rjust(2, '0')
        end

Depende kivanio/brcobranca#259

@OCA-git-bot
Copy link
Contributor

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

@mileo mileo changed the title [IMP] envia instrução de protesto na primeira instrução do bradesco. [14.0][IMP] envia instrução de protesto na primeira instrução do bradesco. Jun 11, 2024
@renatonlima renatonlima requested a review from mbcosta June 12, 2024 16:38
@antoniospneto
Copy link
Contributor

@mileo

No titulo você comentou Bradesco, mas na verdade a correção é pro Itaú, isso ?

@mileo
Copy link
Member Author

mileo commented Jun 18, 2024

Bradesco

Sim, confundi o título.

@mileo mileo changed the title [14.0][IMP] envia instrução de protesto na primeira instrução do bradesco. [14.0][IMP] envia instrução de protesto na primeira instrução do Itau. Jun 18, 2024
mbcosta
mbcosta approved these changes Jun 19, 2024
View reviewed changes
Copy link
Contributor

@mbcosta mbcosta left a comment

Choose a reason for hiding this comment

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

valeu @mileo entendo que é melhor juntar o que foi identificado como igual, mas como pontuei no outro PR #3121 eu estou considerando em alterar esses campos CHAR como esse boleto_protest_code em objetos, a avaliação e opinião sobre isso são bem vindos

@mileo
[IMP] envia instrução de protesto na primeira instrução do bradesco.
148bc39 
lib/brcobranca/remessa/cnab400/itau.rb

        def prazo_instrucao(pagamento)
          return '03' unless pagamento.cod_primeira_instrucao == '09'

          pagamento.dias_protesto.rjust(2, '0')
        end
@mileo mileo force-pushed the fix/14.0-cnab-400-itau-protesto branch from f4b594b to 148bc39 Compare June 19, 2024 20:37
@rvalyi
Copy link
Member

rvalyi commented Jun 19, 2024

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 14.0-ocabot-merge-pr-3120-by-rvalyi-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit ffb9134 into OCA:14.0 Jun 19, 2024
6 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@mbcosta mbcosta mbcosta approved these changes

@renatonlima renatonlima renatonlima approved these changes

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

Successfully merging this pull request may close these issues.
6 participants
@mileo @OCA-git-bot @antoniospneto @rvalyi @renatonlima @mbcosta