Fix the way that we set up the Org template for notes.

[jaccarmac]

Continuation of #327, using the version property of a specific variable rather than the version of Org itself.

[jaccarmac]

I figure if (heaven forbid) Org continues to change this variable in backward-incompatible ways we now have a nice place to set up a cascading cond of version-dependent configuration changes.

[takaxp]

This PR fixes a critical error regarding new definition of org-structure-template-alist, so should be merged at least by Org 9.2 release otherwise many users will confuse.

And the change should also be merged to the stable branch in this repository so that MELPA users can use new code too.

[jaccarmac]

Thanks for commenting on this, looking back I realized I made an error with the version check that might break things in the future. The fixed version of the PR is still critical though, thanks for bumping!

For reference: The version change appeared in c04e357f3d5d93484277a7e439847b1233b872bd. Further discussion of the relevant Org commits is in #327.

[tjsousa]

I'm currently hitting this issue with the latest update to org mode, any idea if it will go forward and released in ox-reveal?

[driftcrow]

(setq org-reveal-note-key-char nil)

[takaxp]

We can find additional related PRs 💯

#359
#361
#362

[takaxp]

Now we can use a new forked repository: https://gitlab.com/oer/emacs-reveal
see also #363

[yjwen]

Done by 1b7957f.
Thanks.

[jaccarmac]

Thanks for that merge. However, I'd like to request that the version check code be amended. This PR uses a very specific way of checking the version, as my local way of installing Org doesn't successfully set org-version. By checking the version on the specific variable as in https://github.com/yjwen/org-reveal/pull/340/files#diff-6d8f0c59126b393965f0226829b47a30R1283, the patch will work regardless of org-version. I can provide another smaller PR if necessary.

[yjwen]

OK, no problem. Have updated according to your suggestion.