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-17894] iOS: Adds beta-reports-active to Entitlements.plist #6266

Merged
merged 1 commit into from Nov 22, 2014

Conversation

FokkeZB
Copy link
Contributor

@FokkeZB FokkeZB commented Oct 24, 2014

Only for App Store builds.

JIRA ticket: https://jira.appcelerator.org/browse/TC-4896

@ingo ingo changed the title Adds beta-reports-active to Entitlements.plist [TIMOB-17894] Adds beta-reports-active to Entitlements.plist Nov 17, 2014
@ingo
Copy link
Contributor

ingo commented Nov 17, 2014

@cb1kenobi who else can review this?

@cb1kenobi
Copy link
Contributor

@ingo Anybody on the iOS team.

@cb1kenobi
Copy link
Contributor

CR'd, looks good. Needs FR.

@ingo ingo changed the title [TIMOB-17894] Adds beta-reports-active to Entitlements.plist [TIMOB-17894] iOS: Adds beta-reports-active to Entitlements.plist Nov 18, 2014
@eric34
Copy link
Contributor

eric34 commented Nov 18, 2014

Not seeing a file "Entitlements.plist" in the ipa. Is this change to the info.plist? Not seeing the key there either.

@FokkeZB
Copy link
Contributor Author

FokkeZB commented Nov 18, 2014

@eric34 If your target is dist-appstore the build/iphone/Entitlements.plist should have the new key.

@eric34
Copy link
Contributor

eric34 commented Nov 18, 2014

OK, So seeing "Entitlements.plist" with correct setting for beta-reports-active based on ad-hoc or app-store builds in the project "build/iphone" folder. iTunes connect set up, and will be conducting test to see if this works with test flight builds. Initially assumed the entitlements must be in the ipa.

@eric34
Copy link
Contributor

eric34 commented Nov 19, 2014

PR: APPROVED
Please note I did see strange behavior from iTunes connect during this test, but feel confident enough for now to approve and then conduct additional (time consuming) tests to help understand the behavior we saw.

What I saw was this:

  1. I created a project using sdk 3.4.1.GA and validated then pushed to iTunes connect. Users had been configured but I saw no App icon, no test flight toggle, and no link on the app version.
  2. I then cleaned the project and built using a custom sdk including this fix. I pushed this to iTunes connect. Same result, no link, no toggle, no icon.
  3. I waited a bit but nothing changed. A bit later I logged out and then logged back in.
  4. At this point, both application versions (1.0.0 - Built with 3.4.1.GA, and 1.0.1 - built with this fix) allowed test flight. I selected 1.0.0 and distributed it to my "test team" (me) and was able to get the notification, and download, install, and launch the application.

So according to theory, the 1.0.0 version of my test application should not have had the entitlement as it was built with 3.4.1.GA. I will need to explore this more, but this will take some time as I will need to prepare two additional applications for upload. It could be that Apple is applying the provision to all versions of the app.

@ingo
Copy link
Contributor

ingo commented Nov 22, 2014

Approved. Merging.

ingo added a commit that referenced this pull request Nov 22, 2014
[TIMOB-17894] iOS: Adds `beta-reports-active` to `Entitlements.plist`
@ingo ingo merged commit 035b211 into tidev:master Nov 22, 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