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

"Add Geolocation (frontmatter) to current note" reproducable failure for any location #98

Closed
MattWilcox opened this issue May 31, 2022 · 2 comments

Comments

@MattWilcox
Copy link
Contributor

MattWilcox commented May 31, 2022

Using "Add Geolocation (frontmatter) to current note" can fail for any location and any note when the note body contains ---.

When using this command, the resulting output is inserted immediately before the last occurance of a --- in the document.

This is wrong, and can result in the information being inserted into the body of the note instead of the frontmatter - if anyone's using that as an in-note separator.

The information should find the second occurance of --- in the note, not the last.

@MattWilcox
Copy link
Contributor Author

Worked out what this was! Made a pull request :)

@esm7
Copy link
Owner

esm7 commented Jun 1, 2022

Thank you! 😎

@esm7 esm7 closed this as completed Jun 1, 2022
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

No branches or pull requests

2 participants