Bug: Template resolution deletes two characters

Question

Bug: Template resolution deletes two characters

jbnv opened this issue 3 years ago · 8 comments

Sorry if this has already been posted. When RoamJS resolves a template, it deletes the two characters before the trigger (jj &c). So the string abcdjjtoday becomes ab[[October 6th, 2021]] instead of abcd[[October 6th, 2021]].

Answer 1 · 2021-10-06T23:05:04.000Z

is this with the smartblocks extension (v2) or with roam42 (v1)

Answer 2 · 2021-10-07T02:29:48.000Z

SmartBlocks extension.

Answer 3 · 2021-10-07T03:52:21.000Z

okay because usually jj is associated with roam42 (v1).

I'm not seeing this on my end. could you post a loom demonstrating the behavior?

Answer 4 · 2021-10-08T00:18:49.000Z

Hmm. I know I changed my trigger configuration in roam/js/smartblocks from xx to jj and somehow it was reset to xx. I removed the old RoamJS code and installed Marketplace, so I shouldn't have v1 left in my graph.

I'm not set up on Loom and I don't want to install software on my machine for just one use. I stated the observed behavior in plain language so I'm not sure how a video of the same thing will add any info.

Answer 5 · 2021-10-08T03:56:07.000Z

I typically ask for videos as there are details that users aren't aware of enough to say in plain text that is caught on video that could illuminate a bug. Either that, or will need more context/repro steps otherwise I can't debug what's going on

I get the sense still that this might be a v1 bug and not a v2 bug. Here's me running v2, notice that "today" is not entered into the input:

Here' me running in v1. The today typing in the input, while not showing the bug here, could sometimes cause inconsistent text clearing bc it's done manually

Answer 6 · 2021-10-09T01:02:37.000Z

The latter behavior is what I'm seeing. I'll see if I can find where the v1 code is still running in my graph.

UPDATE: Checked my roam/js references. Didn't see anything that looked like the old code. Should I try uninstalling Roam42?

Answer 7 · 2021-10-09T01:34:16.000Z

Yea if you see the second one, then Roam42 is installed somewhere. I would try uninstalling Roam42 and try the SmartBlocks extension and see if the bug persists. The former will be replaced by the latter next week

Answer 8 · 2021-10-10T00:06:51.000Z

That took care of it. I'm looking forward to seeing how the rest of the Roam42 features come out in RoamJS.