-
-
Notifications
You must be signed in to change notification settings - Fork 108
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
Docblocks for properties not working #69
Comments
Hi @ricklambrechts, This should be fixed with v2.1.5! |
Awesome! |
@rubenvanassche Packagist is not showing the new version. Could you look into it? |
That's strange, updated it: https://packagist.org/packages/spatie/laravel-settings |
This is not working when using the cache |
Hi @argonzalez2016, could you provide me an example. Casts are only used to transform values between repository and settings class. When caching we serialize the complete setttings object so the values should be exactly the same as before caching. |
I found the issue. I was trying to cache a laravel collection and it was calling the toArray method before serializing. |
Yeah, personally, I wouldn't use too complex types within a settings class because that opens the door for a lot of things that can go wrong. But technically it is possible 😄 |
Hi,
Most of the things are working great, thanks!
But when using the provided option inside a settings class:
I get the exception:
The text was updated successfully, but these errors were encountered: