Skip to content

Updated android to version 5 #208

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Updated android to version 5 #208

wants to merge 1 commit into from

Conversation

narusevic
Copy link
Contributor

@cla-bot cla-bot bot added the cla: yes label Aug 22, 2019
@narusevic narusevic mentioned this pull request Aug 22, 2019
Closed
@narusevic
Copy link
Contributor Author

Build failed, because Encrypted environment variables have been removed for security reasons. See https://docs.travis-ci.com/user/pull-requests/#pull-requests-and-security-restrictions
Any ideas, why this happened?

@elena-p elena-p mentioned this pull request Aug 29, 2019
Merged
@elena-p
Copy link
Contributor

elena-p commented Aug 29, 2019

Hi @narusevic,

Thank you for contributing to nativescript-facebook!
However our CI checks cannot pass when the PR is from fork. This is why I have opened another PR with your changes that we can approve and merge.

@elena-p elena-p closed this Aug 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

cla: yes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@narusevic @elena-p