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

Fix plugin to run on current versions of Obsidian. #3

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

jcfergus
Copy link

This updates the dependencies and code to use the latest Obisdian sample plugin as a base, to solve these errors preventing the plugin from being enabled on recent versions of Obsidian.

i.e. markdown-confluence/markdown-confluence#651

Plugin failure: confluence-integration TypeError: W_e.isDesktopCapturerEnabled is not a function

I have tested it successfully on Obsidian 1.7.4, I don't know about other versions. There are no actual code changes in src/ConfluencePerPageForm.tsx, beyond adding some explicit any types in a few spots to get eslint/tsc to not complain about them - my IDE "fixed" a few indents. I can revert those indentation changes if needed.

@wazeemwoz
Copy link

What needs to be done to get this merged/fixed? Just got obsidian to work with confluence and found the plugin is busted.

@bkahlerventer
Copy link

It appears the project is abandoned, last commit over a year ago.

@gjulianm
Copy link

Considering this project seems to be abandoned, could you @jcfergus post basic instructions on how to install this manually?

@lucasiscovici
Copy link

You can install with BRAT with : https://github.com/lucasiscovici/obsidian-integration/
And version 0.0.1

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 this pull request may close these issues.

5 participants