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-26094] iOS 12: Add new Ti.UI.TextField API's #10088

Merged
merged 15 commits into from Jul 12, 2018

Conversation

hansemannn
Copy link
Collaborator

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

Using no tests until our build machine runs Xcode 10.

@sgtcoolguy
Copy link
Contributor

@hansemannn TiApp.m fails to compile on Jenkins here as well (like it does on #10098). The machine it built on does have Xcode 9 installed...

@sgtcoolguy
Copy link
Contributor

Also this PR looks like it's two separate tickets/issues that aren't particularly related...

@hansemannn
Copy link
Collaborator Author

hansemannn commented Jun 13, 2018

@sgtcoolguy I need to rebase with the latest changes, thanks for merging! The reason these two are together is that all Xcode 10 additions to the SDK require the IS_XCODE_10 macro, which I've put there to unblock development. I will split it up now and we need to rebase once the macro is merged, so Jenkins passes.

EDIT: Moved to #10105

@build
Copy link
Contributor

build commented Jun 13, 2018

Messages
📖

💾 Here's the generated SDK zipfile.

Generated by 🚫 dangerJS

@sgtcoolguy sgtcoolguy changed the title [TIMOB-26094] iOS: Add Xcode 10 developer toolset, add new Ti.UI.TextField API's [TIMOB-26094] iOS: add new Ti.UI.TextField API's Jun 13, 2018
@hansemannn hansemannn changed the title [TIMOB-26094] iOS: add new Ti.UI.TextField API's [TIMOB-26094] iOS 12: Add new Ti.UI.TextField API's Jun 14, 2018
Copy link
Contributor

@vijaysingh-axway vijaysingh-axway left a comment

Choose a reason for hiding this comment

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

CR passed.

@@ -11,7 +11,7 @@
var should = require('./utilities/assertions');

describe('Ti.UI.TextField', function () {
it('.passwordRules', function () {
it.ios('.passwordRules', function () {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thanks Chris!

@build
Copy link
Contributor

build commented Jul 11, 2018

Messages
📖

💾 Here's the generated SDK zipfile.

Generated by 🚫 dangerJS

@hansemannn hansemannn merged commit c13eca8 into tidev:master Jul 12, 2018
sgtcoolguy pushed a commit that referenced this pull request Aug 27, 2018
* [TIMOB-26090] Add Xcode 10 macro, fix compiler warning

* [TIMOB-26094] Expose new autofill constants

* [TIMOB-26094] Expose password rules API

* Update fix-versions

* Remove Xcode 10 related dev-env changes

* Update Titanium_Prefix.pch

* Restructure code

* Update TiUITextWidget.m

* Enforce xcode-10 to run unit tests on Jenkins. Add unit tests for new APIs/constants

* Mark new tests as ios-specific
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

4 participants