louisdh / openterm

OpenTerm, a sandboxed command line interface for iOS
Other
1.63k stars 249 forks source link

Scripts Error 'PridelandDocumentError error 0' #129

Closed jamiejenkins closed 6 years ago

jamiejenkins commented 6 years ago

I have latest beta on an iPad and iPhone and get the error when trying to open up scripts on both. Additionally the script I managed to create before the error started is still there but doesn't show up as when I try to create a new one with name 'test' it says it already exists. I've got it set to iCloud Storage on both devices.

badddd76-f052-4ed1-a76c-3d40caea07a3

jamiejenkins commented 6 years ago

manually deleting the contents of ~/.scripts/ directory seems to fix the issue.

shaun-h commented 6 years ago

I am also seeing this, I created a script on my iPad and I get this error on my iPhone

alessaba commented 6 years ago

Can confirm this bug. I think it has to do with iCloud syncing

alessaba commented 6 years ago

UPDATE: It seems to sync the .scripts folder just fine, and it knows what scripts are in there (i tried re-importing a previously imported example and it says it's duplicated), but it still doesn’t show ant scripts in the view controller.

alessaba commented 6 years ago

Also, running the cub command manually returns Missing file ".scripts/a.prideland/contents/.1.cub.icloud"

Which to me seems like the file hasn’t been downloaded yet from iCloud. Is there a way to download all the .scripts contents from icloud at app launch?

Sent with GitHawk

alessaba commented 6 years ago

Build 35 doesnt seem to have fixed the issue :-/

jamiejenkins commented 6 years ago

On the device where the script is created there is a .1.cub file and the device it is syncing to there is the placeholder .1.cub.icloud file. So I think you are right and it is not actually synching the file data itself. I just tested again on Build 35 as well.

louisdh commented 6 years ago

Thanks for the info, build 36 (available on TestFlight now) should fix this.

alessaba commented 6 years ago

😭GitHawk Upload by filippocld

Sent with GitHawk

louisdh commented 6 years ago

At least the improved error handling works! It's now clear that there's a problem with syncing the metadata.plist from your a.prideland.

jamiejenkins commented 6 years ago

Yes, still an issue on build 36. I deleted everything in .scripts, started app on iPad and imported a script, started app on iPhone and got error when looking at the scripts. On the iPhone it has .metadata.plist.icloud instead of .metadata.plist

louisdh commented 6 years ago

Does build 37 fix it by any chance?

alessaba commented 6 years ago

Yep! It fixed it! Thank you so much 😊

louisdh commented 6 years ago

@jamiejenkins If you can confirm that it's fixed I will close this issue.

jamiejenkins commented 6 years ago

I can confirm that it is working now! 👍🏻

louisdh commented 6 years ago

Awesome, thank you all so much for testing!