Skip to content
This repository has been archived by the owner on Jul 15, 2021. It is now read-only.

Added a simpler way to configure the cache subscriber #27

Merged
merged 8 commits into from
Jun 4, 2015

Conversation

csarrazi
Copy link
Owner

As mentioned in #24: we want to streamline the cache subscriber's configuration (especially the csa_guzzle.cache.adapter configuration.

What we can do for now:

  • Implement a simpler way to configure a cache adapter.
  • Provide backward compatibility.
  • Deprecate the previous way to configure the service.
  • Update the doc for master
  • Add unit tests

This PR will wait for #25 to be merged, in order to prevent merge conflicts on tests.

@csarrazi csarrazi added this to the v1.3.0 milestone May 19, 2015
@csarrazi csarrazi changed the title Added a simpler way to configure the cache subscriber [WIP] Added a simpler way to configure the cache subscriber May 19, 2015
@csarrazi csarrazi changed the title [WIP] Added a simpler way to configure the cache subscriber Added a simpler way to configure the cache subscriber Jun 4, 2015
csarrazi added a commit that referenced this pull request Jun 4, 2015
Added a simpler way to configure the cache subscriber
@csarrazi csarrazi merged commit 231145c into master Jun 4, 2015
@csarrazi csarrazi deleted the deprecate-cache-service branch June 4, 2015 07:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant