Skip to content
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

(feat)capture: add customizable defaults for deleting aborted files #2351

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Hugo-Heagren
Copy link
Contributor

  • org-roam-capture.el (org-roam-capture-delete-aborted-files): new
    custom var
    (org-roam-capture--check-aborted-delete): util function to check
    whether we should delete the file after an aborted capture.
    (org-roam-capture--finalize): use
    org-roam-capture--check-aborted-delete' rather than yes-or-no-p'.

Motivation for this change

I abort captures pretty regularly but I always answer the same to the prompt which asks whether I want to delete the aborted file -- I always want to delete it. This is a (tiny) friction which could be easily customised away. This PR makes that possible.

* org-roam-capture.el (org-roam-capture-delete-aborted-files): new
custom var
(org-roam-capture--check-aborted-delete): util function to check
whether we should delete the file after an aborted capture.
(org-roam-capture--finalize): use
`org-roam-capture--check-aborted-delete' rather than `yes-or-no-p'.
@m-e-leypold
Copy link

Just stumbled over the very same problem. What's keeping this PR from progressing?

@real-or-random
Copy link
Contributor

What's keeping this PR from progressing?

See #2375.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants