search

contentsync / SketchContentSync

Sync sketch files with google docs.
275 stars 17 forks source link

Crashes Sketch (v3.0.0) #56

Closed colllin closed 7 years ago

colllin commented 7 years ago

This could be more of a conflict between 2 plugins... but when I installed this plugin (via SketchRunner), Sketch crashed and I couldn't open any sketch files without crashing the program. Finally I figured out to go to Sketch > Plugins > Manage Plugins... and disabled some plugins until I found that disabling this plugin allowed me to open sketch files again.

I can send a list of my plugins if you think it might be a conflict.

dbrody commented 7 years ago

Hi Collin,

Yes, could you provide details on the Sketch ContentSync plugin version and Sketch Version? The more details you can provide the better we can look into the issue.

Thanks, David

colllin commented 7 years ago

Latest of each.... ContentSync v3.0.0 Sketch v46.1

On Aug 8, 2017, 9:44 AM -0600, David Brody , wrote:

Hi Collin, Yes, could you provide details on the Sketch ContentSync plugin version and Sketch Version? The more details you can provide the better we can look into the issue. Thanks, David — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

contentsync commented 7 years ago

@colllin It looks like it was because we had two folders with the plugin in our release file structure. The new structure now only includes one: SketchContentSync.sketchplugin which should properly install the plugin in SketchRunner.

Version 3.0.2

Hope this helps!

contentsync commented 7 years ago

It looks like we are still seeing some issues. We will continue to investigate.

contentsync commented 7 years ago

It looks like the issue is related to our use of the Keychain to store user account information. Using the keychain keeps your account information safe, outside of Sketch, and easily administerable.

However, it seems that by installing the plugin using SketchRunner, the permissions needed to use the Keychain are invalidated since the system believes SketchRunner is the owner, and not ContentSync.

We are trying to work with SketchRunner to come up with a solution, we will keep this thread updated as we learn more.

For now, please do not install using SketchRunner.

dbrody commented 7 years ago

I've followed this up with #59 as it is more contextual in the title.