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

Add optional created kwarg to djstripe_sync_models management command #1853

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

arnav13081994
Copy link
Collaborator

Added optional created kwarg to the djstripe_sync_models management command to allow users to specify the datetime they would like to initiate sync from. This is far better than always initiating sync from the beginning.

@arnav13081994 arnav13081994 force-pushed the feature/add_created_kwarg_to_models_sync branch from d6a1757 to a39f6d0 Compare December 17, 2022 05:39
@arnav13081994 arnav13081994 force-pushed the feature/add_created_kwarg_to_models_sync branch from 97195f3 to 93cf31e Compare December 18, 2022 13:22
    `created` optional kwarg will allow dj-stripe users to sync models
    after the last sync or after a particulat datetime. This would
    be far better than initiating every sync from the beginning
    every single time.
This approach was taken as it is very difficult to manage
exactly what all models' list calls support the `created`
field. And that is why we run `list()` as is and in case the
given model doesn't support `created` param we catch and remove
it from the kwargs dict and re-run `list()`
@arnav13081994 arnav13081994 force-pushed the feature/add_created_kwarg_to_models_sync branch from 93cf31e to 44baabf Compare December 18, 2022 13:29
@arnav13081994 arnav13081994 self-assigned this Dec 31, 2022
@arnav13081994 arnav13081994 added this to the 2.8.0 milestone Dec 31, 2022
@jleclanche jleclanche force-pushed the master branch 5 times, most recently from 15e04c3 to 8727894 Compare April 25, 2024 14:48
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

1 participant