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-17665] Fixed user location for iOS 8 #48

Merged
merged 3 commits into from Sep 15, 2014

Conversation

pec1985
Copy link
Contributor

@pec1985 pec1985 commented Sep 13, 2014

Details in TIMOB-17665
I'm not sure how to upload the new module.zip. Git is ignoring the zip files

@@ -371,7 +392,7 @@ properties:
map.
description: |
If `true`, the user's location is marked with a pin, and the My Location button will appear in the top
right corner of the screen.
right corner of the screen. Starting in iOS 8, permissions must be added to tiapp.xml. Details in description
Copy link
Contributor

Choose a reason for hiding this comment

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

'in description.'

map.autoresizingMask = UIViewAutoresizingNone;
if ([TiUtils isIOS8OrGreater]) {
Copy link
Contributor

Choose a reason for hiding this comment

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

This condition must be reversed. if (![TiUtils isIOS80orGreater)
Essentially follow current behavior if version < 8

@vishalduggal
Copy link
Contributor

Code Reviewed. Please address comments. Also binary needs to be updated for testing

@pec1985
Copy link
Contributor Author

pec1985 commented Sep 13, 2014

Comments addressed.

@vishalduggal
Copy link
Contributor

Code Reviewed. ACCEPTED

@jonalter
Copy link
Contributor

@pec1985 please include a built zip of the module when you make a change to a module

@ingo
Copy link
Contributor

ingo commented Sep 14, 2014

I believe he was having difficulty doing so, but the .zip file is attached to the ticket.

@jonalter
Copy link
Contributor

FT pass

jonalter added a commit that referenced this pull request Sep 15, 2014
[TIMOB-17665] Fixed user location for iOS 8
@jonalter jonalter merged commit 5fa4669 into tidev:master Sep 15, 2014
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

4 participants