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

allow stopping torrents in verification states #715

Merged
merged 8 commits into from Jan 23, 2022

Conversation

pyrovski
Copy link
Contributor

Fixes #677 #199

Copy link
Member

@ckerr ckerr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's a small tweak mentioned inline that would be nice-to-have before merging this, but otherwise looks good & works as advertised. Thanks for the fix!

libtransmission/rpcimpl.c Outdated Show resolved Hide resolved
@pyrovski
Copy link
Contributor Author

pyrovski commented Jan 5, 2020

Can this be merged?

cfpp2p pushed a commit to cfpp2p/transmission that referenced this pull request Mar 8, 2021
cfpp2p pushed a commit to cfpp2p/transmission that referenced this pull request Mar 8, 2021
@ckerr
Copy link
Member

ckerr commented Oct 13, 2021

I know how long this PR has been in limbo and am working to reduce the project's backlog of older PRs.

@pyrovski is this PR something you'd still like to see land; and if so, could you update the PR to resolve the merge conflicts that it has with transmission/transmission's master branch?

@ckerr ckerr added scope:core needs update The PR has needs to be updated by the submitter labels Oct 13, 2021
@pyrovski
Copy link
Contributor Author

Oops; I think I merged my own master branch instead of that of transmission/transmission. I'll fix that.

@ckerr ckerr added enhancement and removed needs update The PR has needs to be updated by the submitter labels Jan 23, 2022
@ckerr ckerr merged commit 96d9ab5 into transmission:main Jan 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

Request: Add ability to gtk to remove torrent from verification queue.
2 participants