-
Notifications
You must be signed in to change notification settings - Fork 42
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
Add functions to insert IDs or open files based on full text search #127
Conversation
I see now this has been pending for a couple months. Can somebody merge it? Thanks! |
Ah yes, lost track of this. I'll do this shortly! Only need to think of a set of default keybindings... |
replace concat with expand-file-name
Also, I ran into an issue where trying to use The real issue seems be the |
Too bad... :(( |
Now accounts for recursive directories, ie, when deft-recursive is non-nil
The problem was actually in my function |
Also, @AtomicNess123 , if you want to try out these functions prior to a merge, just to see how they're working for you, you can copy them into a scratch buffer, evaluate them, and call them via |
I actually replaced my old zetteldeft.el with the new one and tried the functions and they looked good. Will do more testing today. Thanks! |
|
Strange, since 'full-search' depends on 'find-file'... And both are working fine for me. Could you do 'toggle-debug-on-error', then try 'zetteldeft-find-file-full-title-insert' again, and paste the backtrace here? |
Sure, I tried again, same results.
|
The problem is that the file ("textDocument.txt") doesn't have a zetteldeft ID in the filename. So |
Am I silly or am I silly? :) Thanks! Tried selecting a proper file name and works. |
Finally got around to this, with a terrible delay. These new functions are bound to Thanks for your contribution! |
Fix #29 and #101
cc: @justinmoon @Steven-LANG