KevinBatdorf / obsidian-raycast

Raycast extension with commands for the note taking app Obsidian.
MIT License
106 stars 19 forks source link

Add "Append task" #51

Closed mikejongbloet closed 1 year ago

mikejongbloet commented 1 year ago

Feature idea:

There are two 'quick capture' problems I'm trying to solve - quick capture a note and quick capture a task.

It would be amazing if it were possible to either specify two templates, or if I could set up a second quick capture command somehow.

Do you have any thoughts or guidance?

Thank you

mikejongbloet commented 1 year ago

Thinking a bit more about this, what I'm actually looking to do is "Append to note" where I can specify a string to be appended and a note.

mikejongbloet commented 1 year ago

Hey @marcjulianschwarz -

I've developed this locally on a fork of your repository. The command does the following:

I'm not an engineer, but I kind of know my way around a little.

What I'm not sure about is how to send it to you for consideration to merge in. Raycast suggests I run npx ray publish but I was a bit concerned that would publish the addition without your approval.

I then tried pushing a branch to your repository to open up a Pull Request but got a permission denied message.

Would you be able to point me in the right direction?

p.s. one thing that confused me a little was that the fork I pulled from your git repo didn't have arguments in the JSON manifest. In adding the new command I've also added in arguments for the other tasks in that manifest.

mikejongbloet commented 1 year ago

Uploading a video here of it in action: https://www.youtube.com/watch?v=MrqpTLoICPc

marcjulianschwarz commented 1 year ago

Hi @mikejongbloet,

this looks nice. Could you please push the local branch to your own github profile? You can then open a pull request in the forked repo.

mikejongbloet commented 1 year ago

Hey @marcjulianschwarz…I think I've done that…I've not contributed to a repo before but I followed Githubs forking and PR documentation and I think it's worked.

Let me know.

Thanks Mike

marcjulianschwarz commented 1 year ago

Accepted the pull request.