sawhney17 / logseq-citation-manager

73 stars 12 forks source link

Can't Use Slash Command: Create Inline Literature Note #53

Closed li-chena closed 1 year ago

li-chena commented 1 year ago

Error log from the console:Uncaught (in promise) TypeError: Cannot read properties of null (reading '0')

@Logseq Citations Plugin_software 2022-12-31 22-00-34_new

kmk93posteo commented 1 year ago

For me, "insert inline" and "link to reference" both do not work. All three shortcuts lead me to the selection menu, but after selecting these two options, nothing (useful) happens - I am a layperson, sorry for the bad description. "go to reference" works perfectly fine so I just use it as a workaround.

kerim commented 1 year ago

I'm having this problem as well. Nothing is happening. I am using 0.8.15

sawhney17 commented 1 year ago

Was it working in the previous version? On 6 Jan 2023, 10:51 PM +0400, P. Kerim Friedman @.***>, wrote:

I'm having this problem as well. Nothing is happening. I am using 0.8.15 — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you are subscribed to this thread.Message ID: @.***>

kerim commented 1 year ago

I never had such problems before, but I'm also not sure how frequently the versions have updated and whether or not I used it during each version or not. It has probably been about a month since I used it last.

kerim commented 1 year ago

This seems to be working again for me now. Perhaps fixed in the last Logseq update?

gandalf34 commented 1 year ago

I have the same issue.

gandalf34 commented 1 year ago

https://drive.google.com/open?id=1lqrj0_W--XAmPt6p3JoqwGTm7W1-iqkg

kerim commented 1 year ago

Yeah, this seems broken again with the latest update.

sawhney17 commented 1 year ago

Investigating this.

Sincerely, Aryan Sawhney Homeroom 7.11 On 8 Feb 2023, 10:05 AM +0400, P. Kerim Friedman @.***>, wrote:

Yeah, this seems broken again with the latest update. — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.***>

sawhney17 commented 1 year ago

Have you moved the database to the new location in the assets folder?

Sincerely, Aryan Sawhney Homeroom 7.11 On 8 Feb 2023, 12:35 PM +0400, Aryan Sawhney @.***>, wrote:

Investigating this.

Sincerely, Aryan Sawhney Homeroom 7.11 On 8 Feb 2023, 10:05 AM +0400, P. Kerim Friedman @.***>, wrote:

Yeah, this seems broken again with the latest update. — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.***>

sawhney17 commented 1 year ago

I’ve added a fallback in the latest version, which would prevent the change from breaking workflows.

On 8 Feb 2023, 12:37 PM +0400, Aryan Sawhney @.***>, wrote:

Have you moved the database to the new location in the assets folder?

Sincerely, Aryan Sawhney Homeroom 7.11 On 8 Feb 2023, 12:35 PM +0400, Aryan Sawhney @.***>, wrote:

Investigating this.

Sincerely, Aryan Sawhney Homeroom 7.11 On 8 Feb 2023, 10:05 AM +0400, P. Kerim Friedman @.***>, wrote:

Yeah, this seems broken again with the latest update. — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.***>

gandalf34 commented 1 year ago

Yes it was there at the time of the video

On Wed, Feb 8, 2023, 11:38 Aryan Sawhney @.***> wrote:

Have you moved the database to the new location in the assets folder?

Sincerely, Aryan Sawhney Homeroom 7.11 On 8 Feb 2023, 12:35 PM +0400, Aryan Sawhney @.***>, wrote:

Investigating this.

Sincerely, Aryan Sawhney Homeroom 7.11 On 8 Feb 2023, 10:05 AM +0400, P. Kerim Friedman @.***>, wrote:

Yeah, this seems broken again with the latest update. — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.***>

— Reply to this email directly, view it on GitHub https://github.com/sawhney17/logseq-citation-manager/issues/53#issuecomment-1422227662, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUTAGGJS7U2YGCO3EXYBHHLWWNLPPANCNFSM6AAAAAATNXFJVA . You are receiving this because you commented.Message ID: @.***>

kerim commented 1 year ago

Yes, I moved the database to the new location. Search works, but it won't create anything.

sawhney17 commented 1 year ago

Try the new update.

kerim commented 1 year ago

Seems fixed. Thanks!

sawhney17 commented 1 year ago

Very stupid error. Used a single equal instead of double equals. Good to see it's fixed. On 8 Feb 2023 at 11:08 PM +0400, P. Kerim Friedman @.***>, wrote:

Seems fixed. Thanks! — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.***>