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

fix(ios): fix extension app group and entitlements support (8_0_X) #10814

Merged
merged 4 commits into from Apr 12, 2019
Merged

fix(ios): fix extension app group and entitlements support (8_0_X) #10814

merged 4 commits into from Apr 12, 2019

Conversation

cb1kenobi
Copy link
Contributor

JIRA: https://jira.appcelerator.org/browse/TIMOB-26948

Fixes a typo where it was enabling the app group by setting true instead of 1.

Also correctly reference the entitlements file, specifically when the entitlements is referenced by the CODE_SIGN_ENTITLEMENTS build setting that is a relative path from the extension project's root.

@build build added this to the 8.0.1 milestone Mar 28, 2019
@build
Copy link
Contributor

build commented Mar 28, 2019

Warnings
⚠️

iphone/cli/commands/_build.js#L6434 - iphone/cli/commands/_build.js line 6434 – 'out' is defined but never used. (no-unused-vars)

Messages
📖

💾 Here's the generated SDK zipfile.

📖

✅ All tests are passing
Nice one! All 3023 tests are passing.

Generated by 🚫 dangerJS against b896f4d

@janvennemann janvennemann changed the title fix(ios): Fixed extension app group and entitlements support. fix(ios): fix extension app group and entitlements support (8_0_X) Mar 29, 2019
@sgtcoolguy
Copy link
Contributor

Awaiting @janvennemann 's approving review from #10813

@keerthi1032
Copy link
Contributor

FR Passed. App group showed and enabled correctly and entitlements files listed correctly.
Name = Mac OS X
Version = 10.13.6
Node.js
Node.js Version = 8.9.1
npm Version = 5.5.1
Titanium CLI
CLI Version = 5.1.1
Studio =5.1.2.201903111843
Titanium SDK
SDK Version = local 8.0.1 sdk and 8.1 master sdk
Cli =7.0.10

@keerthi1032 keerthi1032 merged commit 7185393 into tidev:8_0_X Apr 12, 2019
@cb1kenobi cb1kenobi deleted the TIMOB_26948_8_0_X branch June 7, 2019 16:02
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.

None yet

5 participants