Skip to content
This repository was archived by the owner on Feb 7, 2019. It is now read-only.

Conversation

@lini
Copy link
Contributor

@lini lini commented Jan 2, 2018

  • before prepare hook updated to handle new structure of platforms/android directory and copy the google-services.json file to the new location
  • travis CI updated to use correct bundle build command instead of the old "npm run build-xxx-bundle"

@lini lini self-assigned this Jan 2, 2018
@lini
Copy link
Contributor Author

lini commented Jan 2, 2018

Fixes #165.
Builds on the previous PR for the issue #169

.travis.yml Outdated
node_js: "6"
jdk: oraclejdk8
script: cd demo && npm run build.plugin && npm i && npm run build-ios-bundle
script: cd demo && npm run build.plugin && npm i && tns build ios --bundle
Copy link
Contributor

Choose a reason for hiding this comment

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

It would be nice if we add --env.uglify preparing the plugin to become verified one

.travis.yml Outdated
script: >
echo { \"project_info\": { \"project_number\": \"424050927453\", \"firebase_url\": \"https://pptest-53916.firebaseio.com\", \"project_id\": \"pptest-53916\", \"storage_bucket\": \"pptest-53916.appspot.com\" }, \"client\": [ { \"client_info\": { \"mobilesdk_app_id\": \"1:424050927453:android:21d2dd8987e1e3d1\", \"android_client_info\": { \"package_name\": \"org.nativescript.ppTest\" } }, \"oauth_client\": [ { \"client_id\": \"424050927453-o7gv6bcmknujjoaaoa5r4d452cutb9jf.apps.googleusercontent.com\", \"client_type\": 3 } ], \"api_key\": [ { \"current_key\": \"AIzaSyDr3fqH7Z8-vhBNpAges_KFTA7oOoylGvU\" } ], \"services\": { \"analytics_service\": { \"status\": 1 }, \"appinvite_service\": { \"status\": 1, \"other_platform_oauth_client\": [] }, \"ads_service\": { \"status\": 2 } } } ], \"configuration_version\": \"1\"} > demo/app/App_Resources/Android/google-services.json &&
cd demo && npm run build.plugin && npm i && npm run build-android-bundle
cd demo && npm run build.plugin && npm i && tns build android --bundle
Copy link
Contributor

Choose a reason for hiding this comment

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

--env.uglify --env.snapshot are nice additions here

@lini lini merged commit eadaea2 into master Jan 3, 2018
@lini lini deleted the lini/add-ns-3.4.0-compatibility branch January 3, 2018 09:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants