Skip to content

Conversation

@ronnyjohnti
Copy link
Contributor

✅ Descrição do propósito desse Pull Request


🧭 Referência a Issue


❓ O que foi feito para atingir isso?


🏃‍♀️ Tipo de mudança

Marque as opções relevantes:

  • Bug fix (correção de bug)
  • Nova feature (mudança não retrocompatível que adiciona funcionalidade)
  • Mudança de breaking (correção ou feature que faria com que a funcionalidade existente não funcionasse como esperado)
  • Documentação (somente mudanças ou atualizações na documentação)

🕵️ Como foi testado?

  • Critério de aceitação
  • Testes de software (TDD, BDD, UNITÁRIO, INTEGRAÇÃO, E2E)

Checklist: ✔️

  • Meu código segue as diretrizes do projeto
  • Eu fiz um code review com minha equipe
  • Eu comentei meu código, especialmente em áreas de difícil entendimento
  • Eu atualizei a documentação correspondente
  • Testes novos e existentes passaram localmente com minhas alterações

Observação:

@ronnyjohnti ronnyjohnti self-assigned this Jun 10, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces a RabbitMQ consumer to send notification emails when opinions are published.

  • Adds an AMQP service and console command to consume the opinions_published queue and dispatch emails
  • Implements a PublishedOpinions Mailable and Blade template for the email content
  • Updates Docker supervisor, config/app, and example/testing env files to wire up the new queue

Reviewed Changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated no comments.

Show a summary per file
File Description
resources/views/emails/published-opinions.blade.php New email template for "PublishedOpinions"
docker/php-fpm/supervisord.conf Supervisor program entry to run the new RabbitMQ consumer
docker/php-fpm/99-xdebug.ini Switched Xdebug mode to coverage
config/app.php Registered opinions_published queue and added exchanges config key
composer.json Moved and deduplicated the ext-pdo requirement
app/Services/AmqpService.php Introduced AmqpService for queue declaration and consumption
app/Mail/PublishedOpinions.php Created the PublishedOpinions Mailable
app/Console/Commands/ConsumePublishedOpinions.php Added command to consume messages and send notification emails
.env.testing Set testing values for RabbitMQ exchange and queue
.env.example Changed default DB to pgsql and added new RabbitMQ variables

@ronnyjohnti ronnyjohnti force-pushed the feature/add-published-opinions branch from 073adf5 to 40def67 Compare June 11, 2025 17:16
Copy link
Member

@jeff-doliveira1 jeff-doliveira1 left a comment

Choose a reason for hiding this comment

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

top!

@ronnyjohnti ronnyjohnti merged commit 41cab4c into main Jul 7, 2025
@ronnyjohnti ronnyjohnti deleted the feature/add-published-opinions branch July 7, 2025 13:08
@ronnyjohnti ronnyjohnti restored the feature/add-published-opinions branch July 10, 2025 15:03
@ronnyjohnti ronnyjohnti deleted the feature/add-published-opinions branch July 10, 2025 15:05
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.

4 participants