-
Notifications
You must be signed in to change notification settings - Fork 182
fix: Bug obsidian tags not being added for regex notes #230
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
Conversation
|
Is this included in the current version? @st3v3nmw . |
|
It's not currently included. |
Also, will I lose the settings of the plugin if I make a complete change in the plugin folder? Can't I just fix the part where you have modified the code? From comparison, I could see 2 places where there were a semicolon and has been removed in the new code... |
|
Try deleting the package-lock.json file and try again.
No. You'll only change the Also, I've made a release on my fork that you can easily download and use. |
|
Thanks!! :)
…On Mon, 31 May 2021 at 10:52, Stephen Mwangi ***@***.***> wrote:
Try deleting the package-lock.json file and try again.
Also, will I lose the settings of the plugin if I make a complete change
in the plugin folder?
No. You'll only change the main.js file. The settings are stored in
data.json.
Also, I've made a release on my fork that you can easily download and use.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#230 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AQMZMQMQOTUS5GU4AJKUNKDTQMMKHANCNFSM4ZCDEUVQ>
.
|
|
bump - would really love this functionality! |
|
Why is this not being merged? Just made a similar pull request (#287) that does not include changes to gitignore or the lockfile, if that's the problem. I closed my pull request because I found out about this one. |
Because the developer is away since the last release :) |
|
Thanks @st3v3nmw ! This has been most frustrating up until now. Much appreciated. I cloned your fork but hopefully it will be merged soon. |
I replaced with the one you released. It seems not working.. |
|
@Pseudonium - is this something that is goint to be implemented soon? |
620828f to
a27de96
Compare
|
E2E tests for this fix are done in #479 Though its late, Thanks for the bugfix ! |
|
The pull request is merged, why the "Tags" seem to only work on the same line of answer? I tried the same regex as the issue #196: #tag_up
Q: question 1 #tag_question
A: answer example 1 #tag_answer
#tag_above
<!--ID: 1757264898513-->
Tags: #tag_up
Q: question 2 Tags: #tag_question
A: answer example 2 Tags: #tag_answer
<!--ID: 1757264898508-->
Tags: #tag_above
Tags: #tag_up
Q: question 3 Tags: #tag_question
A: answer example 3 #tag_answer
<!--ID: 1757264907731-->
Tags: #tag_above |

This should fix #196 where Obsidian tags in regex notes aren't added to Anki.