search

nvim-neorg / neorg-telescope

Telescope.nvim integration for Neorg
GNU General Public License v3.0
187 stars 24 forks source link

Some calls are erroring out #57

Closed davidbgonz closed 5 months ago

davidbgonz commented 6 months ago

First noticed that the find_neorg_files call was throwing error. Going though all the calls, I am getting the below errors for some methods when I call them in the cmdline. Hopefully these are helpful to see a pattern. I know recently there's been a lot of changes with telescope and neorg.

find_aof_tasks: known issue: this can be ignored image

find_norg_files: image

switch_workspace: Telescope popup appears, but then after a few seconds this error shows image

find_context_tasks: known issue: this can be ignored image

find_project_tasks: known issue: this can be ignored image

find_aof_project_tasks: known issue: this can be ignored image

Rest of the calls are working for me. Let me know if there is additional information you need from my setup. Everything is setup/up-to-date through lazy. I'm on v8.x of neorg with the luarocks integration with lazy.

austinbutler commented 6 months ago

Possibly related: https://github.com/nvim-neorg/neorg-telescope/pull/58

benlubas commented 6 months ago

@davidbgonz all of the task pickers are known to be broken, they're listed as such in the readme.

The other two were fixed shortly after this issue went up, if you still have problems you can leave them here or open a new issue but otherwise this one can be closed

davidbgonz commented 6 months ago

Thanks for letting me know on the task picker calls. I'll update the original message to mark that.

I still have the above errors running the find_norg_files and switch_workspace calls.

benlubas commented 6 months ago

ah, try out #58, should fix that

davidbgonz commented 6 months ago

Yep, the changes in that PR works for me. All the calls are functional again. I appreciate you taking the time helping me out. This is one of those extensions that you don't realize how much you lean on it until it starts throwing errors. I'll close this issue out and in the meantime I will patiently wait for that PR to be merged.

benlubas commented 6 months ago

You can (and probably should) leave it open until the pr is merged.

pysan3 commented 5 months ago

It is now merged. I think we can close this (I don't have the permission).

davidbgonz commented 5 months ago

Thanks for the PR! Closing this now.