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 seeders and provider back to the torrent info #72

Closed
wants to merge 2 commits into from

Conversation

purple-emily
Copy link
Collaborator

Closes #52

@purple-emily
Copy link
Collaborator Author

image

@purple-emily purple-emily marked this pull request as ready for review February 5, 2024 17:19
@iPromKnight
Copy link
Collaborator

The problem is atm we've no service that's updating seeders.
So they'll only ever show what they are on the initial ingestion

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 * * * *"

@purple-emily
Copy link
Collaborator Author

Do you wish to hold this merge until thats done?

@iPromKnight
Copy link
Collaborator

No we can add it if you'd like 😃
It's just they'll all say 100 as the dmm ones are hardcoded lol

@purple-emily
Copy link
Collaborator Author

So open an issue for seeder updates & link this issue?

@iPromKnight
Copy link
Collaborator

Aye good idea

Sorry I'm out atm 😃

@davidjameshowell
Copy link
Contributor

No we can add it if you'd like 😃 It's just they'll all say 100 as the dmm ones are hardcoded lol

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?

@purple-emily
Copy link
Collaborator Author

#73 this is a good idea. Tagging the issue here for clarity.

@purple-emily purple-emily force-pushed the add-seeders-and-provider-information branch from ddceead to feaf6e4 Compare February 8, 2024 09:02
@Gabisonfire
Copy link
Collaborator

Feels like this one should be closed, @purple-emily ?

@purple-emily purple-emily deleted the add-seeders-and-provider-information branch March 14, 2024 13:42
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.

Add number of seeders on the streams page
4 participants