CommunityScripts icon indicating copy to clipboard operation
CommunityScripts copied to clipboard

Fix syntax error in timestampTrade.py

Open Written2001 opened this issue 1 year ago • 5 comments

This line leads to an syntax error: "unmatched '['" due to the usage of double quotes for both the f-string and the inner dictionary key. The proposed solution fixes that.

Written2001 avatar Sep 24 '24 18:09 Written2001

Nested quotes are supported in Python 3.12 and above

Maista6969 avatar Sep 24 '24 18:09 Maista6969

Not sure if it's the default but the python version in my stash docker container is 3.11.2

Written2001 avatar Sep 24 '24 20:09 Written2001

Ah. Nerethos is still on py3.11.2

feederbox826 avatar Sep 26 '24 00:09 feederbox826

Another bug that needs to be fixed: the "path" setting is not configured in the defaults. The UI has this as "Download parent folder" but it is not set until it is modified so it needs to be added to the settings default object. Could you also add the following to the settings on line 1335 "path": "/download_dir/tt-auto",

Tweeticoats avatar Sep 26 '24 02:09 Tweeticoats

Ah. Nerethos is still on py3.11.2

Yeah the Image is based on debian bookworm which has no prebuild deb packages for python3.12

Written2001 avatar Sep 26 '24 08:09 Written2001

superceded by https://github.com/stashapp/CommunityScripts/commit/3e4fb6383e8a405ad5ed2b1d47502c75bd8d42de

feederbox826 avatar Nov 27 '24 06:11 feederbox826