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

12: fix ttl vs options obj #16

Merged
merged 2 commits into from
Apr 5, 2019
Merged

12: fix ttl vs options obj #16

merged 2 commits into from
Apr 5, 2019

Conversation

cvburgess
Copy link
Owner

Apollo's setter uses an options object with a TTL prop, therefore, we should match that.

closes #15
closes #12

@cvburgess cvburgess mentioned this pull request Apr 5, 2019
@cvburgess cvburgess merged commit 410b33b into master Apr 5, 2019
@cvburgess cvburgess deleted the ttl-options branch April 5, 2019 18:53
@cvburgess cvburgess mentioned this pull request Apr 24, 2019
# 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.

Fix ttl Cache TTL doesn't seem to work
2 participants