search

marcusolsson / obsidian-projects

Plain text project planning in Obsidian
Apache License 2.0
1.42k stars 56 forks source link

Fix for Obsidian 1.6.x compatibility #873

Closed Acylation closed 3 months ago

Acylation commented 4 months ago

Will merge and release this after 1.6.x becomes public. Before that we will have a nightly build that need to install manually for Obsidian insiders.

marcusolsson commented 4 months ago

Is this a breaking change? Do we need to update versions.json?

Acylation commented 4 months ago

It's expected to work on previous versions of Obsidian after merging this. The only problem is we changed the icon for open in new window. So I think it's fine to keep the minAppVersion.

marcusolsson commented 4 months ago

Awesome! 🚀

GitMurf commented 4 months ago

Will merge and release this after 1.6.x becomes public. Before that we will have a nightly build that need to install manually for Obsidian insiders.

@Acylation / @marcusolsson have you considered using BRAT for delivery of nightly builds / beta testing? It is super easy to setup and a big part of Obsidian ecosystem is using it these days to test plugins and deliver beta versions etc.

Here are details: https://tfthacker.com/brat-developers#manifest-beta.json

marcusolsson commented 4 months ago

@GitMurf Love BRAT! Didn't know about the manifest-beta.json file though. I've added it to the CI now, so every merged PR will create a beta release now. Let me know if it doesn't work 🙏

GitMurf commented 4 months ago

@marcusolsson worked like a charm. Nice work and thank you!

Acylation commented 3 months ago

Merge upon Obsidian 1.6.2 becomes public.