-
Notifications
You must be signed in to change notification settings - Fork 154
feat: Add basic Category Support for qBittorrent #902
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
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: Inochi Amaoto <[email protected]>
Signed-off-by: Inochi Amaoto <[email protected]>
Signed-off-by: Inochi Amaoto <[email protected]>
872e960
to
9aebf88
Compare
I'll merge this PR with squash, please do not force-push |
Sorry, I will do not do this again. Thanks for your reminder. |
@@ -19,6 +19,7 @@ export enum TorrentPriority { | |||
export interface TorrentProperties { | |||
bytesDone: number; | |||
comment: string; | |||
category: string; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
are we sure this is not nullable or defined?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it can be nullable, as ''
can be used to identify no category.
@@ -88,6 +88,14 @@ export const getContextMenuItems = (torrent: TorrentProperties): Array<ContextMe | |||
{ | |||
type: 'separator', | |||
}, | |||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not very familiar with our frontend code, is it possible to only show this with qBittorrent?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Me too, I think it is necessary for showing with qBittorrent only.(Although I have found the category is not shown if every torrent does not have category)
Description
Introduce basic category support for qBittorrent.
These PR still have some gaps:
Related Issue
#686
Screenshots
Types of changes