Skip to content

Conversation

danilopolani
Copy link
Contributor

@danilopolani danilopolani commented May 29, 2025

Added

  • Events count on Logs table
  • Events details in Detail modal

Fixed

  • "Created at" event
  • Message ID without brackets to match webhooks data

@danilopolani danilopolani added this to the Webhooks milestone May 29, 2025
@danilopolani danilopolani self-assigned this May 29, 2025
@danilopolani danilopolani requested a review from Copilot May 30, 2025 07:42
Copy link

@Copilot 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 pull request adds an events list to the log detail view and enhances various parts of the logging functionality. Key changes include:

  • Renaming the timestamp key in webhook log creation from createdAt to created_at.
  • Updating the log query on the list page to eager load events.
  • Adding a new events_count column and events display in the log resource table.
  • Minor updates to email logging and front-end tab functionality.

Reviewed Changes

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

Show a summary per file
File Description
src/Webhooks/Actions/ProcessWebhook.php Updated event log creation to use consistent snake_case.
src/Resources/LogResource/Pages/ListLogs.php Eager loading of events added to the log query.
src/Resources/LogResource.php Added events_count column and formatted events display in table.
src/Models/LogEvent.php Added created_at to fillable attributes.
src/Models/Log.php Updated events relation to return latest events.
src/Jobs/SendMailJob.php Modified email message ID handling for MailGun compatibility.
resources/views/modals/details.blade.php Added Events tab and refactored tab management logic.
composer.json Updated PHPStan analysis command with increased memory limit.

@danilopolani danilopolani merged commit 6b518fd into master May 30, 2025
11 checks passed
@danilopolani danilopolani deleted the feat/events-visualization branch May 30, 2025 07:43
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.

1 participant