feat: add manual torrent adding #785
Conversation
|
@dreulavelle @filiptrplan It downloads the correct thing, but it doesn't seem to symlink it. I'm adding a new symlink event at the end, what could the problem be? |
src/controllers/items.py
Outdated
|
|
||
| session.commit() | ||
|
|
||
| request.app.program.em.add_event(Event("Symlinker", item)) |
There was a problem hiding this comment.
we handle by id's now, so when the id_handling branch is done this will need to change to item._id
There was a problem hiding this comment.
Ah this still needs fixing I believe
src/controllers/items.py
Outdated
| @@ -385,7 +458,8 @@ class SetTorrentRDResponse(BaseModel): | |||
| def add_torrent(request: Request, id: int, magnet: str) -> SetTorrentRDResponse: | |||
| torrent_id = "" | |||
| try: | |||
| torrent_id = add_torrent_magnet(magnet) | |||
| _, infohash = get_type_and_infohash(magnet) | |||
| torrent_id = add_torrent(infohash) | |||
There was a problem hiding this comment.
This will fail for AllDebrid :/
There was a problem hiding this comment.
I think that we should refactor some things related to the manual torrent adding when we merge #762 so we have a standard interface for the downloaders.
There was a problem hiding this comment.
I fully agree!
There was a problem hiding this comment.
This needs looked at again.
davidemarcoli
force-pushed
the
feature/manual-torrent-adding
branch
2 times, most recently
from
c71adee
to
51b64e4
Compare
davidemarcoli
force-pushed
the
feature/manual-torrent-adding
branch
from
27ac133
to
c80a064
Compare
dreulavelle
changed the title
filiptrplan
force-pushed
the
feature/manual-torrent-adding
branch
from
c18faed
to
5e33c9c
Compare
chore: remove not needed state transition
davidemarcoli
force-pushed
the
feature/manual-torrent-adding
branch
from
5e33c9c
to
27d5643
Compare
Pull Request Check List
Resolves: #issue-number-here
Description: