-
Notifications
You must be signed in to change notification settings - Fork 73
Add support for Laravel 5.7 #28
Comments
Feedback please: 573112a. 😁 |
I'm testing this right now. I have already commented an issue in the README file. |
I found an issue with the commit 573112a, the encrypted fields are being correctly generated but an unencrypted value is being stored in the database. I've debugged the code and the issue is related to this change introduced 21 days ago (which in fact breaks support for Laravel 5.6 also). Previously, on insert the The result is that the package "is broken" because it doesn't achieve it's purpose. My proposal is to change the |
@axelitus Can you make a separate issue for this and include code to reproduce the bug? Without digging in too much, this seems accurate and would explain other oddities when using 5.6/5.7. |
I'll open an issue later on with a use case |
Closing in lieu of #30. |
Currently this package cannot be installed with Laravel 5.7 due to the restriction:
Please add support for Laravel 5.7
The text was updated successfully, but these errors were encountered: