You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
First of all, thank you for this wonderful package! Why this is not an opt-in solution our of the box with Laravel, I don't know.
I'm currently ignoring the errors in hope of seeing dwindling usage numbers of that Browser (Safari 9), but it seems like people just don't upgrade…
Since it seems to not support the "let" keyword, would it be possible to change the template to use "var" instead? This should also cover IE 10 and the like. I can also provide a PR if you're currently out of time. Cheers!
Edit: Just saw that there's already a PR open for this… #86
The text was updated successfully, but these errors were encountered:
Expected Behavior
No Errors
Acutal Behavior
Hundreds of Client-Side Errors (collected via Sentry)
Environment
Stack Trace
SyntaxError: Unexpected identifier 'lastCheck'
(https://github.com/GeneaLabs/laravel-caffeine/blob/master/resources/views/script.blade.php#L2)First of all, thank you for this wonderful package! Why this is not an opt-in solution our of the box with Laravel, I don't know.
I'm currently ignoring the errors in hope of seeing dwindling usage numbers of that Browser (Safari 9), but it seems like people just don't upgrade…
Since it seems to not support the "let" keyword, would it be possible to change the template to use "var" instead? This should also cover IE 10 and the like. I can also provide a PR if you're currently out of time. Cheers!
Edit: Just saw that there's already a PR open for this… #86
The text was updated successfully, but these errors were encountered: