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-20337] Parity: enabling run-on-main-thread #7878
Conversation
Shouldn't we still support and deprecate the old way first? |
It's ok. When it was introduced earlier, it was already mentioned that it's experimental so there should not be any project dependencies on this. |
Would be cool to just throw a log-warning once and remove that for 6.0.0 so everyone know's what's going on. |
Sounds reasonable. Will update pr |
@@ -35,6 +34,7 @@ | |||
<property name="ti.bb.invoke.target.key.push" type="string">ti.testapp.invoke.push</property> | |||
<property name="ti.bb.invoke.target.key.open" type="string">ti.testapp.invoke.open</property> | |||
<property name="push_title" type="string">Some Title for BB Push, typically the app name"</property> | |||
<property name="run-on-main-thread" type="bool">true</property> | |||
<code-processor> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah 👍
@@ -2076,7 +2076,14 @@ iOSBuilder.prototype.initialize = function initialize() { | |||
// Remove the debugHost/profilerHost check when we have debugging/profiling support with JSCore framework | |||
// TIMOB-17892 | |||
this.currentBuildManifest.useJSCore = this.useJSCore = !this.debugHost && !this.profilerHost && (this.tiapp.ios['use-jscore-framework'] || false); | |||
this.currentBuildManifest.runOnMainThread = this.runOnMainThread = (this.tiapp['run-on-main-thread'] === true); | |||
// Remove this check on 6.0.0 | |||
if(this.tiapp.ios && (this.tiapp.ios.hasOwnProperty('run-on-main-thread'))) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there should a space between if
and (
Looks good. Approved. |
JIRA: https://jira.appcelerator.org/browse/TIMOB-20337