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

Error consistently printing to console #130

Open
codyburleson opened this issue Jan 9, 2024 · 4 comments
Open

Error consistently printing to console #130

codyburleson opened this issue Jan 9, 2024 · 4 comments

Comments

@codyburleson
Copy link

Console error consistently printing to the console:

app.js:1 Failed to open view TypeError: Cannot read properties of undefined (reading 'navFileContainerEl')
    at t.eval (plugin:alx-folder-note:4821:45)
    at t.wrapper [as load] (plugin:alx-folder-note:4107:20)
    at t.<anonymous> (app.js:1:1402700)
    at app.js:1:237258
    at Object.next (app.js:1:237363)
    at app.js:1:236279
    at new Promise (<anonymous>)
    at v (app.js:1:236024)
    at t.open (app.js:1:1402572)
    at t.<anonymous> (app.js:1:1825868)
@FynnFreyer
Copy link

Can confirm, have the same thing happening.
Seems to be due to fileExplorer.dom being undefined.

I have added appropriate checks #133

@FynnFreyer
Copy link

With that patch applied, I don't get any errors, and the plugin seems to be working still. Haven't done extensive testing though (and also won't, except for just using it normally)

@MJHop
Copy link

MJHop commented Feb 4, 2024

I'm sorry for the stupid question.. but how do I add in this fix? Having the same issue and I uninstalled and reinstalled the plugin but it's still happening.

@SebastianMC
Copy link

Hi @MJHop

It's been more than a month since your post here, but maybe this will be helpful also for others.

Please look at my longer comment in issue 134. It contains instructions on how to easily install the patched version of alx-folder-note plugin, compatible with Obsidian 1.5.4 (and newer) by using the well known BRAT plugin.

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

Successfully merging a pull request may close this issue.

4 participants