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: wrong website url when using a custom directory for WP install #528

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

gutobenn
Copy link

@gutobenn gutobenn commented Jan 3, 2025

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

Este PR corrige a URL usada na configuração inicial do plugin quando se usa um diretório de instalação diferente da URL do site.
Por exemplo: numa instalação do Bedrock, a URL do wordpress é https://www.dominio.com.br/wp e a URL do site é https://www.dominio.com.br. Ao fazer a configuração inicial do plugin do pagar.me, o correto seria que os webhooks e a URL de retorno sejam configurados para /wc-api/pagarme-hub, e não /wp/wc-api/pagarme-hub, que é como está hoje.

Cenários testados

Instalação de WordPress usando o Bedrock.
Não tenho certeza se este PR gera algum problema em outros tipos de instalação

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.

@gutobenn gutobenn changed the title Fix: wrong website url when using a custom directory for WP install fix: wrong website url when using a custom directory for WP install Jan 3, 2025
@fabiano-mallmann fabiano-mallmann requested a review from a team January 6, 2025 14:45
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.

2 participants