Taapeli / isotammi-addons

Gramps addons for Isotammi project / Isotammen Gramps-lisäosia
8 stars 2 forks source link

Request saving the "Capture output" setting in the SuperTool Script notes. #37

Open emyoulation opened 2 months ago

emyoulation commented 2 months ago

Could the "Save As Note" include the "Capture output" setting ?

[filter]

[expressions]

[scope]
selected

[unwind_lists]
False

[commit_changes]
False

[summary_only]
True
kkujansuu commented 2 months ago

Currently the "Capture output" setting is not stored anywhere. It is preserved within a session - even if another script is loaded or the database is switched. But it is reset every time Supertool is started.

I couldn't decide if the setting shoud be stored in the script file (or note) or if it should be a global config setting that would be preserved across sessions. In a way, I think it should not be tied to the script but be a "user setting".

emyoulation commented 2 months ago

I think excluding the setting creates an exception. Exceptions make SuperTool more difficult to document, learn and remember.

Right now, we have to post-process the Script (or "SuperTool Script" note) to remove the Category where it is desired to be more generic or not be overridden.

Also, the Capture Output is going to be desireable for specific script... but it will go unused generally. So saving with the script seems reasonable.

kkujansuu commented 2 months ago

OK, makes. sense.

su 14. heinäk. 2024 klo 20.10 Emyoulation or BAMaustin ( @.***) kirjoitti:

I think excluding the setting creates an exception. Exceptions make SuperTool more difficult to document, learn and remember.

Right now, we have to post-process the Script (or "SuperTool Script" note) to remove the Category where it is desired to be more generic or not be overridden.

Also, the Capture Output is going to be desireable for specific script... but it will go unused generally. So saving with the script seems reasonable.

— Reply to this email directly, view it on GitHub https://github.com/Taapeli/isotammi-addons/issues/37#issuecomment-2227416706, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEIYOXWASKKHOSG65X2PJC3ZMKWHXAVCNFSM6AAAAABK3HRK2GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRXGQYTMNZQGY . You are receiving this because you commented.Message ID: @.***>