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

Give user option to do initial sync and prevent loading synced history #47

Open
rafaelgomesxyz opened this issue Nov 4, 2020 · 1 comment · May be fixed by #319
Open

Give user option to do initial sync and prevent loading synced history #47

rafaelgomesxyz opened this issue Nov 4, 2020 · 1 comment · May be fixed by #319
Assignees
Labels
enhancement New feature or request

Comments

@rafaelgomesxyz
Copy link
Member

This would be very nice to prevent unnecessary requests to the Trakt API, specially since they're imposing limits now. Here's how I think it should happen:

  1. Upon opening the sync page for a streaming service for the first time, the user is prompted with an option to sync their entire history so far (since the Trakt API allows 1000 requests every 5 minutes, it's perfect for scenarios like this)
  2. We store the date of this initial sync (let's call it date A)
  3. In the future, every time the user opens the sync page for that same streaming service again, we make sure that only history newer than date A is loaded
  4. Additionally, every time we detect that the user has synced all history from date A to a date B, we update date A with date B

Not only would this save a ton of requests, but I believe it would also enhance the user experience, since they would only see items that they haven't synced. What do you think?

@MrMamen
Copy link
Member

MrMamen commented Nov 4, 2020

That makes sense. Just have to make sure that it's possible to override or reset.

@rafaelgomesxyz rafaelgomesxyz self-assigned this May 3, 2021
@rafaelgomesxyz rafaelgomesxyz added the enhancement New feature or request label May 3, 2021
This was referenced Jul 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants