Skip to content

Create Symfony 6.4 example app #166

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

Merged
merged 10 commits into from
May 16, 2025
Merged

Conversation

mclack
Copy link
Contributor

@mclack mclack commented May 18, 2023

Goal

To document BugSnag being used in a Symfony 6 app. Symfony 6.4 is the only maintained version of Symfony v6, and we currently have no ^6.x examples.

Design

Followed the current Symfony integration guide along with the symfony50 example app.

Changeset

symfony62 added to the example directory. (Will need to be updated to symfony64)

Testing

Tested each route configured in DefaultController.php - each exception was reported to the BugSnag dashboard successfully.

@mclack mclack marked this pull request as draft May 19, 2023 12:52
@mclack mclack changed the title Create Symfony 6.2 example app Create Symfony 6.4 example app Jan 13, 2025
@mclack mclack marked this pull request as ready for review May 14, 2025 13:28
@mclack mclack requested a review from agrzegorzewski May 14, 2025 13:29
@mclack
Copy link
Contributor Author

mclack commented May 16, 2025

@agrzegorzewski I've tested this with the Symfony server and with Docker - both seem to work fine with all the routes. This LGTM, so if you can approve I'll merge it.

Copy link
Contributor

@agrzegorzewski agrzegorzewski left a comment

Choose a reason for hiding this comment

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

LGTM

@mclack mclack merged commit df24965 into next May 16, 2025
12 checks passed
@mclack mclack deleted the SUP-3062/create-symfony-62-example branch May 16, 2025 13:32
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