chrisgrieser / shimmering-obsidian

Alfred Workflow with dozens of features for controlling your Obsidian vault.
https://alfred.app/workflows/chrisgrieser/shimmering-obsidian/
MIT License
830 stars 41 forks source link

[Bug] scratchpad_append_prefix doesn't pass datetime placeholder #112

Closed joshsarvesh closed 1 year ago

joshsarvesh commented 1 year ago

Feature Requested

I'm on version 3.5.3.

Bug Description At the moment I setup the workflow for my first vault, appending a text to scratchpad fails to write datetime prefix specified in scratchpad_append_prefix with a value "- {date:yyyy-MM-dd}, {time:HH:mm} ".

Result in the scratchpad file looks like:

Relevant Screenshot No response

To Reproduce No response

Debugging Log: [16:09:21.274] Shimmering Obsidian[Script Filter] Queuing argument '(null)' [16:09:26.801] Shimmering Obsidian[Keyword] Processing complete [16:09:26.802] Shimmering Obsidian[Keyword] Passing output 'test' to Arg and Vars [16:09:26.803] Shimmering Obsidian[Arg and Vars] Processing complete [16:09:26.803] Shimmering Obsidian[Arg and Vars] Passing output 'test' to Run Script [16:09:27.011] Shimmering Obsidian[Run Script] Processing complete [16:09:27.018] Shimmering Obsidian[Run Script] Passing output '/MasterNotes/DailyJournal/scratchpad ' to Conditional [16:09:27.019] Shimmering Obsidian[Conditional] Processing complete [16:09:27.020] Shimmering Obsidian[Conditional] Passing output '/MasterNotes/DailyJournal/scratchpad ' to Call External Trigger [16:09:27.021] Shimmering Obsidian[External] Processing complete [16:09:27.022] Shimmering Obsidian[External] Passing output '/MasterNotes/DailyJournal/scratchpad ' to Run Script [16:09:27.687] Shimmering Obsidian[Run Script] Processing complete [16:09:27.694] Shimmering Obsidian[Run Script] Passing output '' to Hide Alfred [16:09:27.695] Shimmering Obsidian[Run Script] Passing output '' to Conditional [16:09:27.696] Shimmering Obsidian[Conditional] Processing complete [16:09:27.698] Shimmering Obsidian[Conditional] Passing output '' to Run Script

Relevant Screenshot

SCR-20230124-md9 SCR-20230124-me7

Checklist

chrisgrieser commented 1 year ago

Sorry, a previous version removed support for the dynamic placeholders from Alfred, since a necessary restructuring of the code made it infeasible to support Alfred placeholders. Unfortunately, it would require an disproportionate amount of time to re-implement the feature, so it is dropped for now. Again, sorry!

I only forgot to remove it from the settings, which has been fixed in the latest release.