search

toshism / org-linker-edna

GNU General Public License v3.0
38 stars 6 forks source link

Prepend IDs with "id:" to make them navigable #5

Closed novoid closed 4 years ago

novoid commented 4 years ago

Starting with org-edna version 1.1.2, this is a valid syntax:

** Test heading
:PROPERTIES:
:TRIGGER: ids("id:foo" "id:bar") todo!(NEXT)
:END:

** Foo heading
:PROPERTIES:
:ID: foo
:END:
** Bar heading
:PROPERTIES:
:ID: bar
:END:

This has the advantage, that the IDs within the ids() property are clickable links to their destination headings. I'd love to see those id: prefixes added by org-linker-edna so that dependencies are also navigable links.

toshism commented 4 years ago

So, I'm not actually totally clear exactly how triggers with multiple ids should really be formatted. If you want to try out the change I just pushed let me know if it's correct.

novoid commented 4 years ago

I'm sorry that I was using an example with multiple targets, which is not the main purpose of this ticket. The main goal is to use ids(id:foo) instead of ids(foo) or - for non-UUID items as outlined in #4 ids("id:foo") instead of ids("foo").

I newly pulled from GH and can now confirm this issue solved. Thanks!

novoid commented 4 years ago

Just a remark: the id: prefix also works for UUIDs. So if you're using UUIDs, you probably also want to have those prefixes there. I don't so it's not relevant to me personally.