-
Notifications
You must be signed in to change notification settings - Fork 40
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 seeders and provider back to the torrent info #72
Conversation
The problem is atm we've no service that's updating seeders. It's not the responsibility of the consumer. Its sole purpose is ingestion of torrent data from the queue. We need a new separate service that handles updates of seeders periodically Ideally this would be wrapped in pm2 in the container as it allows execution of a task on a cron schedule ala: pm2 start update-seeders.js --cron "*/15 * * * *" |
Do you wish to hold this merge until thats done? |
No we can add it if you'd like 😃 |
So open an issue for seeder updates & link this issue? |
Aye good idea Sorry I'm out atm 😃 |
Would it be better to put a non-positive value or a different placeholder there to indicate that it is not currently accurate except on ingest? |
#73 this is a good idea. Tagging the issue here for clarity. |
ddceead
to
feaf6e4
Compare
Feels like this one should be closed, @purple-emily ? |
Closes #52