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

improve: allow integration in the default scope #352

Merged
merged 4 commits into from
Jul 23, 2024

Conversation

RafaMelazzo
Copy link
Contributor

@RafaMelazzo RafaMelazzo commented Jul 10, 2024

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

  • Permitir integração no escopo Default, retornando a loja para o escopo de origem da integração
  • Integração em escopo de 'Website' é removida do banco de dados ao marcar a opção 'Use default' nas configurações
  • Removido a opção de integração do escopo 'Store View'

Cenários testados

  • Loja retornando ao escopo de origem após integração
  • Compras sendo realizadas na account correta quando há mais de uma integração:
    • Integração em diferentes accounts no escopo 'Default' e 'Website 1', sem integração no escopo 'Website 2'
    • Compra no 'Website 1' realizada na account do 'Website 1'
    • Compra no 'Website 2' realizada na account do escopo 'Default'
  • Registros do banco de dados removidos após salvar as configurações com a opção 'Use default' marcada no escopo 'Website 1'
  • Opção 'Integração Pagar.me' não aparece nos escopos de 'Store View'

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 marked this pull request as ready for review July 17, 2024 21:20
Copy link

@RafaMelazzo RafaMelazzo merged commit 4a4101c into stg Jul 23, 2024
6 checks passed
@RafaMelazzo RafaMelazzo deleted the PAOPN-391/improve-allow-default-integration branch July 23, 2024 18:30
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