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

Inclusão do Texto de dezenas do MS Nota Premiada #1575

Closed
wants to merge 3 commits into from

Conversation

SuportecTL
Copy link
Contributor

Imprime o texto conforme LEI Nº 5.463, DE 16 DE DEZEMBRO DE 2019.

Decreto

@joaoroyosilva
Copy link

o MS nota premiada é obrigatório aqui no MS.

@AgnaldoSilva0 AgnaldoSilva0 self-requested a review March 5, 2025 14:20
@AgnaldoSilva0
Copy link
Contributor

Olá, boa tarde, @SuportecTL, tudo bem?

Primeiramente gostaria de agradecer pela sua contribuição. 🚀

Realizei os testes do seu PR e está funcionando corretamente.
Porém, notei que você enviou novos commits para a sua branch master, o que acabou incluindo implementações que não são relacionadas à Nota Premiada.

Você poderia criar um novo PR contendo apenas os commits referentes à Nota Premiada?

Os commits são:
5f047e9
9ff5ae5


Uma outra alteração que gostaria que você fizesse é em relação ao nome da variável dezenas (Imagem 1).
Imagem 1:
image
Seria interessante usar algo como: mensagem ou mensagemAdicional.
Motivo: Essa tag não se refere somente as dezenas, pode ser usado para outras observações também.

Assim que abrir o novo PR com os ajustes solicitados, irei revisar e fazer o merge.

Novamente, agradecemos a contribuição. 💯

@AgnaldoSilva0
Copy link
Contributor

Bom dia,

Devido a ausência de resposta em tempo hábil, foi feito os ajuste solicitados e vinculado os commits desse PR no PR alternativo: #1578

Novamente obrigado a todos pela contruição.

@AgnaldoSilva0 AgnaldoSilva0 removed their request for review March 12, 2025 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants