Skip to content
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

Allow passing update options with caches #27

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Floriferous
Copy link
Contributor

This PR makes it possible to fine-tune the collection updates made by denormalize.

In our case, we use the option getAutoValues: false and valide: false, to avoid having caches trigger our simple schema autovalues.

The main one being our updatedAt dates, because of this package they never really meant anything, because writing on another collection, would trigger a cache in the concerned collection and update its updatedAt field.

Care should be taken if you need the cache in one of your autoValues, our internal policy is that we should never create such an autoValue :)

@lgtm-com
Copy link

lgtm-com bot commented Dec 10, 2021

This pull request introduces 1 alert when merging 9e3c819 into d3fc1c0 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

@StorytellerCZ StorytellerCZ added this to the v0.7 milestone Dec 31, 2022
@StorytellerCZ
Copy link
Member

@Floriferous could you please update with latest and take a look on the issue raised by LGTM?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants