Skip to content

Update Init for Facebook #389

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 3 commits into from
Closed

Conversation

nitrag
Copy link

@nitrag nitrag commented Nov 25, 2016

Updating for swift 3

hramos and others added 2 commits May 23, 2016 17:05
@facebook-github-bot

This comment has been minimized.

@nitrag

This comment has been minimized.

@facebook-github-bot

This comment has been minimized.

Parse.setApplicationId("parseAppId", clientKey:"parseClientKey")
PFFacebookUtils.initializeFacebookWithApplicationLaunchOptions(launchOptions)
// AppDelegate.swift (Swift 3)
func application(application: UIApplication, didFinishLaunchingWithOptions launchOptions: [NSObject: AnyObject]?) -> Bool {
Copy link
Contributor

@hramos hramos Nov 28, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is > necessary here? It should render fine within the code block, and it looks like you didn't need to use this encoding later in the same file.

for some reason > was encoded to >
@TomWFox
Copy link
Contributor

TomWFox commented Mar 8, 2019

I'm sorry that this has been over looked. Would you be able to update this code for swift 4.2 (+ the latest version of the Parse iOS SDK) and then I will take another look?

Also would you be able to remove the vertical parameter alignment as it doesn't appear well on the docs:
screenshot 2019-03-08 at 11 58 12

@TomWFox TomWFox self-assigned this Mar 8, 2019
@TomWFox TomWFox changed the base branch from master to gh-pages March 12, 2019 14:50
@TomWFox
Copy link
Contributor

TomWFox commented Mar 12, 2019

I'm closing this now as I have started a new pull request (#594) that brings in these changes (those of which are still needed).

@TomWFox TomWFox closed this Mar 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants