Skip to content

Feature/drafts load flag #679

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

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

Conversation

edeustace
Copy link
Member

@edeustace edeustace commented Jul 12, 2017

when querying for item drafts use the expires constraint so that we only load drafts whose expires date is in the future.

This feature can be disabled by setting the env var: LOAD_EXPIRED_ITEM_DRAFTS to true. This will return the app to the legacy behavior of not using expires as a query constraint.

The feature flag is only temporary. Once this has run on production and there has been no issues, then remove the flag and default always to using the expires constraint. Once this is in place we can start to clean up the drafts collection.

This feature can be disabled by setting the env var: `LOAD_EXPIRED_ITEM_DRAFTS` to true. This will return the app to the legacy behavior of not using `expires` as a query constraint.

> The feature flag is only temporary. Once this has run on production and there has been no issues, then remove the flag and default always to using the `expires` constraint. Once this is in place we can start to clean up the drafts collection.
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.

1 participant