-
-
Notifications
You must be signed in to change notification settings - Fork 27
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
Add support for ignoring columns in the database #62
Conversation
Hey @grantholle, thanks for your PR. I'll try and take a proper look and test things out over the weekend. Would you mind reverting the test names to keep the |
I would going to use the |
We can drop support for PHP 7.4, that works for me and I imagine the majority of users of the package. |
Yeah I agree. Since v2.0.0 is already tagged and is php ^8.0, might as well just go with that major version. I think you can still support the older versions of Laravel, but just bump the php requirement. |
Ok @andreaselia, should be all set now |
Thanks @grantholle, apologies about the delay. |
This adds support to ignore some columns when saving the page view record, i.e. ignoring the
host
column since I have a single tenant app and don't need it to bloat the table.I also changed the test function names to get rid of the deprecation warning about metadata.