Skip to content
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

README: More detailed Android building instructions #11306

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

KreAch3R
Copy link

I tried to build the android app and experienced a lot of problems which I solved after a lot of trial and error, so I thought others can skip them.

Another tip I didn't know how exactly to include it, so I'll leave it here: If you get errors about node-sass not getting installed, change your NodeJS version to v11 (v11.15.0) using nvm: https://github.com/nvm-sh/nvm.

I tried to build the android app and experienced a lot of problems which I solved after a lot of trial and error, so I thought others can skip them. 

Another tip I didn't know how exactly to include it, so I'll leave it here: If you get errors about `node-sass` not getting installed, change your NodeJS version to v11 (v11.15.0) using nvm: https://github.com/nvm-sh/nvm.
@KreAch3R
Copy link
Author

Why would you close this request, copy my commit and change the author to you, removing my ownership, and then pushing again to the official repo @gabrielbazan7 ? Why would this be merged @cmgustavo ? This is not good practice in any open source project I know. Copy/force-push the commit to whatever secondary unofficial repo you want but keep the ownership.

@gabrielbazan7
Copy link
Collaborator

Sorry. It's been open for a long time, and for some reason tests failed. I will re open it.

@gabrielbazan7 gabrielbazan7 reopened this Feb 15, 2021
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.

2 participants