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

helm-org-insert-link-to-heading-at-marker now supports ID properties #26

Closed
wants to merge 1 commit into from
Closed

Conversation

rodrigo-morales-1
Copy link

@rodrigo-morales-1 rodrigo-morales-1 commented Jul 10, 2021

Solves #25 and is a shorter rewriting of #23.

This have been accomplished by rewriting the function in order for it
to use org-store-link and org-insert-last-stored-link. Thus, the user
is able to decide whether headlines must be referenced through their
IDs (by setting org-id-link-to-org-use-id to t) or through their
names.

When submitting a pull request, please include the following information:

  • Emacs versions tested with: GNU Emacs 27.2
  • Org versions tested with: 9.4.4 (release_9.4.4 @ /usr/share/emacs/27.2/lisp/org/)
  • helm versions tested with: 20210324.1927
  • helm-core versions tested with: 20210704.556

Note that Emacs and Org versions persist "in the wild" for some time after release, so it is not appropriate to only test with the latest released or development versions of Org; tests must include versions still commonly in use. Proposed changes must not break functionality for existing users.

This have been accomplished by rewriting the function in order for it
to use org-store-link and org-insert-last-stored-link. Thus, the user
is able to decide whether headlines must be referenced through their
IDs (by setting org-id-link-to-org-use-id to t) or through their
names.
@alphapapa
Copy link
Member

@rdrg109 Please look carefully at the code you have removed. You appear to have omitted some important parts that set the buffer. I don't think the code you wrote will work correctly in all cases.

@alphapapa alphapapa added the enhancement New feature or request label Jul 13, 2021
@alphapapa alphapapa self-assigned this Jul 13, 2021
@rodrigo-morales-1 rodrigo-morales-1 closed this by deleting the head repository Apr 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants