Skip to content
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

TIMOB-9878 Update to latest version of FB's API for continued functional... #2530

Merged
merged 2 commits into from Jul 12, 2012

Conversation

WhichKatieDid
Copy link
Contributor

...ity. Tests should include Kitchen Sink, #3240 #8127 #3169 and #3211, and possibly what happens when the user changes their password on another computer at the same time.

@sptramer
Copy link
Contributor

See comment on jira regarding forward-compatibility. Note that this will not prevent review/acceptance of this PR.

@sptramer
Copy link
Contributor

Code reviewed. Looks OK. APPROVED.

@sptramer
Copy link
Contributor

Functional test (project build). Failed:

/Volumes/Landslide/src/appcelerator/titanium_mobile_origin/iphone/iphone/../Classes/FBConnect/Facebook.m:108:32: error: use of undeclared identifier 'TiUtils'
                appSupportsBackgrounding = ![TiUtils boolValue:@"UIApplicationExitsOnSuspend" properties:[[NSBundle mainBundle] infoDictionary] def:NO];
                                             ^
1 error generated.
/Volumes/Landslide/src/appcelerator/titanium_mobile_origin/iphone/iphone/../Classes/FBConnect/FBLoginDialog.m:95:40: error: use of undeclared identifier 'kRedirectURL'
                        [[url absoluteString] rangeOfString:kRedirectURL].location != NSNotFound) {
                                                            ^
1 error generated.

Indicates that some of our custom code may have been dropped or inappropriately diffed.

@WhichKatieDid
Copy link
Contributor Author

Ready for review again.

@sptramer
Copy link
Contributor

Code reviewed, APPROVED.

@sptramer
Copy link
Contributor

OK, seeing an inconsistency during testing that I can't confirm is a regression or not since we can't even log in with the older SDK - when I log out and then log back in, there is no additional prompt for a login. There are some other issues (probably related to the facebook SDK): Sometimes the login dialog does not appear (remains activity indicator) - but this is very intermittent.

The login/logout issue seems like a problem though. Can anyone confirm if this is a regression?

EDIT: This was a bizarre circumstance created by having an old set of user defaults for Kitchen Sink sitting around, apparently.

@WhichKatieDid
Copy link
Contributor Author

I've been noticing that FB has been sluggish in some spots, but a lot of that is their own SDK. Do you think we could compare against the facebook app?

@sptramer
Copy link
Contributor

Continued testing, iPhone Sim 5.1:

KS -> Facebook: Pass
9878: Pass (note: when you log out in this app, the login is not persistent; strange, but means that is likely not a regression)
8127: Pass
3211: Pass
3169: Part of KS (pass)
3240: Pass

APPROVED

sptramer pushed a commit that referenced this pull request Jul 12, 2012
TIMOB-9878 Update to latest version of FB's API for continued functional...
@sptramer sptramer merged commit 93e10b1 into tidev:master Jul 12, 2012
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.

None yet

2 participants