Skip to content

Conversation

@tgpetrov
Copy link
Contributor

@tgpetrov tgpetrov commented Aug 6, 2018

Due to mismatch in the angular versions when the facebook plugin is manually built and referenced in the demo-angular app, it crashes

@ghost ghost assigned tgpetrov Aug 6, 2018
@ghost ghost added the new PR label Aug 6, 2018
Copy link
Contributor

@VladimirAmiorkov VladimirAmiorkov left a comment

Choose a reason for hiding this comment

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

We should add the .metadata.json to the .gitignore file and remove those as they are auto generated and should not be under source control. Like this https://github.com/NativeScript/nativescript-fresco/blob/master/.gitignore#L53

@tgpetrov
Copy link
Contributor Author

tgpetrov commented Aug 7, 2018

@VladimirAmiorkov, you're right, I added them to the .gitignore

@tgpetrov tgpetrov merged commit 86df303 into master Aug 7, 2018
@ghost ghost removed the new PR label Aug 7, 2018
@tgpetrov tgpetrov deleted the tgpetrov/update-ng-deps branch August 7, 2018 06:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants