-
Notifications
You must be signed in to change notification settings - Fork 34
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
Create org-mode links to elfeed entries #18
base: master
Are you sure you want to change the base?
Conversation
(org-add-link-type "elfeed" 'org-elfeed-open) | ||
(add-hook 'org-store-link-functions 'org-elfeed-store-link) | ||
|
||
(defun org-elfeed-open (path) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would you be so kind to rename these function to make them consistent with the naming of the other functions?
org-elfeed-open
-> rmh-elfeed-org-open
org-elfeed-store-link
-> rmh-elfeed-org-store-link
Thank you for submitting your PR's! Could you add a line to the README to describe how these changes can be used? I also have some small comments concerning the naming of function. If you fix them I'll merge it as soon as I've tested the changes. |
@@ -195,6 +196,41 @@ all. Which in my opinion makes the process more traceable." | |||
"Load all feed settings before elfeed is started." | |||
(rmh-elfeed-org-process rmh-elfeed-org-files rmh-elfeed-org-tree-id))) | |||
|
|||
(org-add-link-type "elfeed" 'org-elfeed-open) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you perform these setup function in the `elfeed-org`` autoload function?
Elfeed now includes orgmode links in elfeed-link. |
This permits the creation of org-mode links from elfeed entries and jump to it.