-
-
Notifications
You must be signed in to change notification settings - Fork 436
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
Added support for Laravel Ignition #3954
Conversation
For those who want to try it immediately on latest v20.6.0
add to your
|
yep I don't think this "save configuration" works at all |
I started a discussion on how to better implement the integration: |
Hi there 👋 Ignition creator here! It's cool that you're considering using our error page in Magento. I notice that this merge was already reverted. If I can help in any way with integration this in Magento let me know! All questions welcome. |
Hi @freekmurze! Thank you for your awesome package! As developer of the package We've also opened a PR to list our module among the compatible platforms: |
Hi @freekmurze it's an honor to have you here, as @empiricompany was saying we created a much better solution to integrate Ignition in OpenMage in an external module , supporting Flare and OpenAI suggestions ;-) |
@empiricompany I can give you a coupon so you can use Flare for free. Send a mail to [email protected] if you'd like that coupon 👍 |
I think Magento 2 could also benefit from having Ignition as its error page. Unfortunately, I'm not that familiar with the Magento community where I should pitch that idea. If you think the maintainers of Magento 2 might be interested, feel free to send them my way should they have questions. |
@freekmurze Magento2 already has this: https://github.com/swissup/module-ignition ;-) |
Laravel Ignition is a pretty famous module for a better handling of error pages: https://flareapp.io/docs/ignition/introducing-ignition/overview
I thought it could be interesting for developers to have built-in, but absolutely optional (it's a pretty big package) in openmage.
The logic is:
If DeveloperMode is disabled or the spatie-ignition package is not installed, OM behaves as usual.
It shouldn't introduce any performance degradation.
This is the error page for a PHP error (function doesn't exist):
This is the error page for a default PHP exception:
And this is a Mage::throwException:
How to test
composer require "spatie/ignition"
(as of 24 april 2024 you also needcomposer require laravel/serializable-closure
)