fix: remove legacy remember_last_path config and update docs#174
Merged
stabldev merged 3 commits intostabldev:mainfrom Nov 20, 2025
Merged
fix: remove legacy remember_last_path config and update docs#174stabldev merged 3 commits intostabldev:mainfrom
stabldev merged 3 commits intostabldev:mainfrom
Conversation
stabldev
requested changes
Nov 20, 2025
Owner
stabldev
left a comment
There was a problem hiding this comment.
Thank you @mukundkumarjha
Can you also update the test case? I forgot this part to mention,
torrra/tests/test_core_config.py
Line 69 in 9959ab3
| torrra config set general.download_in_external_client true | ||
| ``` | ||
|
|
||
| This would change the `remember_last_path` setting to `false`. Note that boolean values (`true`/`false`) should be provided as such, and strings should be enclosed in quotes if they contain spaces or special characters (though for simple paths, it might not always be necessary depending on your shell). |
Owner
There was a problem hiding this comment.
here just change the keyword remember_last_path to download_in_external_client. lets keep the rest of the content for more clarity. also quotes are not needed. pls remove them.
Contributor
Author
|
@stabldev now I get it I have updated the test case. 👍 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
fixes #173