Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add support for Laravel cache #7310

Merged
merged 4 commits into from
Jan 10, 2023
Merged

Add support for Laravel cache #7310

merged 4 commits into from
Jan 10, 2023

Conversation

vpominchuk
Copy link
Contributor

@vpominchuk vpominchuk commented Dec 24, 2022

Description:

This PR adds support for Laravel cache to the library.

Laravel provides Cache facade to easily operate with any configured cache engine (Redis, Memcached,...). Laravel cache engine is configuring using Laravel config files, so we don't need any configuration at this level.

Review

Added support for Laravel cache.
Documentation updated
@sgiehl
Copy link
Member

sgiehl commented Jan 9, 2023

@sanchezzzhak are you familiar with laravel and can maybe have a look at that one?

@sanchezzzhak
Copy link
Collaborator

sanchezzzhak commented Jan 9, 2023

I'm sure this code will work for the latest version.
If a person wants to use earlier versions of frameworks, this code may not work(

@vpominchuk pls fix PHPCS check
php vendor/bin/phpcbf for local repository

@sanchezzzhak sanchezzzhak enabled auto-merge (squash) January 10, 2023 09:12
@sanchezzzhak sanchezzzhak merged commit 513efdd into matomo-org:master Jan 10, 2023
@justinvelluppillai justinvelluppillai changed the title Laravel cache Add support for Laravel cache Jan 12, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants