📙 Updates needed to iOS Parse UI docs

[TomWFox]

The iOS Parse UI docs are in need of updates. Generally consisting of simple syntax changes and fixes.

I have got the ball rolling:

• Merged Ios update signup ui #595 which addresses changes suggested in Fix Swift sample for SignUpViewController fields #368
• Merged Update code snippets for sign up & log in view controller #597 updates all the swift code for the PFSignUpViewController and PFLogInViewController
• Merged Update swift code snippet for PFTableViewCell #600 updated the swift code for PFTableViewCell
• Merged Update swift code snippets for PFQueryTableViewController #599 updated the swift code for PFQueryTableViewController

[TomWFox]

The link to the API Reference for PFSignUpView currently 404s - I can seem to find the API Reference for Parse UI.

This is the current link: http://parseplatform.org/Parse-SDK-iOS-OSX/api/Classes/PFSignUpView.html

[noobs2ninjas]

I'm going over this now and I had a few questions. Not wanting to make that pull request super long I thought I'd ask them here. Heres the first one I found that I think may needed updating.

The Local Datastore

... To enable the datastore, add libsqlite3.dylib

I dont think this is really relevant anymore is it? Its been years since I've had to manually add libraries. Also, just curious, does local data store still use sqlite 3?

[TomWFox]

Not sure, I believe it does still use sqlite3 - if you use the iOS simulator you can go into the file system and find the sqlite file

[noobs2ninjas]

Gotcha!

I figured the library is still being used. However, I don't think users have to manually add it as instructed in the document since Xcode 7 or 8. Which means the minimum iOS version Parse supports requires an Xcode version that automatically compiles included libraries found in the source code.

[noobs2ninjas]

The next thing I found is this:

If you need to refresh an object you already have with the latest data that is in the Parse Cloud, you can call the fetch method like so:

myObject.fetch()

Not only would this not compile because in swift it requires a try but all other platforms examples always show preference to "fetchInBackgroundWithBlock" type functions as asynchronous tasks are generally preferred when doing potentially long-running tasks as opposed to by this example would be blocking on the main thread.

Agreed?

If so I think I can handle the rest. Just want to make sure that consistency between documentation is preferred.

[TomWFox]

Ok, I’m not sure on the automatic compiling, have you tested it?

Changing to fetchinbackground seems logical, and yes consistency between docs is definitely preferred.

[noobs2ninjas]

I just downloaded the Parse source and it has zero linked frameworks and only Bolts is mentioned on target dependencies to build the Parse-iOS target itself. Ooo I also just found that the dylib at the end means its a dynamic library which Xcode will detect and build itself on all Xcode versions supporting our minimum target (iOS 8)

[noobs2ninjas]

This is weird. Just doesnt seem like it belongs in a doc about Parse IMO. Thats just general programming best practices.

We recommend that you NameYourClassesLikeThis and nameYourKeysLikeThis, just to keep your code looking pretty.

[noobs2ninjas]

I just noticed this example in the Retrieve section of Local Data Store

`PFQuery *query = [PFQuery queryWithClassName:@"GameScore"];
[query fromLocalDatastore];
[[query getObjectInBackgroundWithId:@"xWMyZ4YE"] continueWithBlock:^id(BFTask *task) {
if (task.error) {
// Something went wrong.
return task;
}

// task.result will be your game score
return task;
}];`

Might be just my opinion but in order to avoid confusion I'm thinking we should stick to examples that use PFObjectResultBlock and other PFQuery response typealias' as we don't cover Bolts(BFTask) anywhere in this documentation and this example may be confusing.

Edit: I also just noticed the unnecessary return task lol

[noobs2ninjas]

One more thing. Probably should have noticed this sooner. Seems like there is a lot of redundant documentation in Objects and Local Data Store. Thoughts?