search

GENL / matice

Use your Laravel translations in JavaScript.
MIT License
100 stars 16 forks source link

Respect additional JSON paths from Laravel #48

Closed redelschaap closed 3 weeks ago

redelschaap commented 1 month ago

Laravel allows a developer to register additional folders that contain JSON based translation files.

The Matice package did not respect these additional folders, causing a difference in the translations loaded server side and client side.

This PR adds support for additional JSON paths since Laravel 8.

I also added Laravel 9 and additional PHP versions to the test suite. The current test suite does not support Laravel 10 or up because the PHPUnit configuration syntax is too old for the PHPUnit version used since Laravel 10. If it's fine to drop support for Laravel <= 8, I suppose it would be easier to write tests for newer Laravel versions.

redelschaap commented 3 weeks ago

@GENL We would really like to use this. Could you please review my PR? If there is anything I can do, let me know.