Skip to content

Change the JavaScriptImplementation attribute to utilize the require path#403

Merged
slavchev merged 1 commit intomasterfrom
JavaScriptImplementation-Require
Mar 30, 2016
Merged

Change the JavaScriptImplementation attribute to utilize the require path#403
slavchev merged 1 commit intomasterfrom
JavaScriptImplementation-Require

Conversation

@jasssonpet
Copy link
Copy Markdown
Contributor

@jasssonpet jasssonpet commented Mar 25, 2016

@jasssonpet jasssonpet added this to the 2.0.0 (Under Review) milestone Mar 25, 2016
@ns-bot
Copy link
Copy Markdown

ns-bot commented Mar 25, 2016

💚

@slavchev
Copy link
Copy Markdown

👍
ping @atanasovg @blagoev @Plamen5kov

@atanasovg
Copy link
Copy Markdown
Contributor

👍
IMO this way the functionality is more intuitive to the JS developer. We just need to describe that the calling directory in this case is the logical app root.

@jasssonpet
Copy link
Copy Markdown
Contributor Author

I wonder if the static binding generator should be changed in any way? I see that the attribute is used here: https://github.com/NativeScript/android-static-binding-generator/blob/51fca6c633c51e5f03bb31050c3508638dac53e1/project/generator/src/org/nativescript/staticbindings/Generator.java#L204

@slavchev
Copy link
Copy Markdown

I am already working on static binding generator and I'll do the changes.

@Plamen5kov
Copy link
Copy Markdown
Contributor

👍 looks great and I believe it's more intuitive

@slavchev slavchev merged commit ae24ee2 into master Mar 30, 2016
@Plamen5kov Plamen5kov removed this from the 2.0.0 milestone Apr 18, 2016
@slavchev slavchev deleted the JavaScriptImplementation-Require branch April 19, 2016 13:10
@jasssonpet jasssonpet self-assigned this Apr 27, 2016
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.

5 participants