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

fix: recipient document maxLength #322

Merged
merged 2 commits into from
Mar 27, 2024
Merged

Conversation

RafaMelazzo
Copy link
Contributor

Git Merge

Important

Certifique-se de criar o PR para a branch develop.

Qual o tipo de PR é esse? (marque todos os aplicáveis)

  • Refatoração
  • Adição de funcionalidade
  • Correção de bug
  • Otimização
  • Atualização de documentação

Descrição

  • Corrigido alternação de maxLength no campo do número de documento quando document_type é corporation.
  • Corrigido título da página de recebedores quando entra para visualizar.

Cenários testados

Não foi possível efetuar testes, pois o erro não ocorre no meu ambiente local. Mas o recurso de maxLengh não é necessário já que o campo possui máscara, o que já limita o seu tamanho.

Copy link

⚠️ Este repositório ainda não está catalogado no Backstage. ⚠️

Por favor, catalogue-o seguindo as instruções nesta documentação. [Via VPN].

💁 Qualquer problema ou dúvida, estamos no Slack, basta abrir um ticket no canal #help-foundation-platform.

@devsec-app-pagarme
Copy link

Gandalf - Continuous AppSec

📌 Lembrete

Este repositório está sendo monitorando de forma automática e contínua em busca de achados que possam comprometer a segurança da aplicação.
Para maiores detalhes, acesse aqui à plataforma.

📋 Resumo de achados no repositório magento2

Criticade Achados
Critical 0
High 6
Medium 0
Low 0

@RafaMelazzo RafaMelazzo changed the base branch from develop to stg March 27, 2024 20:26
Copy link

sonarcloud bot commented Mar 27, 2024

@RafaMelazzo RafaMelazzo merged commit f999d0f into stg Mar 27, 2024
5 checks passed
@RafaMelazzo RafaMelazzo deleted the PAOPN-681/new-recipients-fields branch March 27, 2024 20:31
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