Skip to content

fix: set sha safely#52

Open
okv wants to merge 1 commit intosilvanocerza:mainfrom
okv:main
Open

fix: set sha safely#52
okv wants to merge 1 commit intosilvanocerza:mainfrom
okv:main

Conversation

@okv
Copy link
Copy Markdown

@okv okv commented Mar 28, 2026

Hi @silvanocerza, thank you very much for your work on this amazing Obsidian Plugin!

I find it very useful especially when using on mobile. Unfortunately, sometimes mobile sync was failing for me with this error:

Error syncing. TypeError: Cannot set properties of undefined (setting 'sha')

Eventually, I added guards in three places, installed and tested the plugin using BRAT.

Please take a look at this patch when you have some time.

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.

1 participant