-
Notifications
You must be signed in to change notification settings - Fork 4
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
Lib Auth Facebook #3
Comments
Ola @Bernardoow utilizei este plugin: http://ngcordova.com/docs/plugins/facebook/ |
Sim quando vc add algum plugin vc digita npm install plugin -S |
Ou, ainda, quando dá um ionic state restore a parte dos plugins é restaurada a partir da chave cordovaPlugins |
Fabio, a galera deu ideia para não enviar os plugins.. por esse motivo do package.json.. Nele fica as informações das libs que o projeto deve ter. Ao clonar o projeto vc da um npm install que as libs são automaticamente baixadas. |
@Bernardoow sabe se da pra colocar os parâmetros do plugin no package também? |
Podia ter uma boa alma pra fazer essa issue e abrir o PR :) |
@fabiorogeriosj não acho que seja uma boa colocar os parâmetros no package.json, porque na verdade você nunca deveria versionar isso. Nunca se versiona valores de configuração, por questões de segurança e de facilidade na hora de montar o ambiente. O que eu costumo fazer: Exemplo: e faço minha aplicação ler o valor do config.json e coloco no README.md as explicações das chaves Com isso, quem baixar o projeto tem o config.json.sample para usar de base e só precisa trocar os valores delas, de acordo com o seu ambiente/configuração. |
Legal @chrisbenseler! Neste caso os parâmetros são o idapp e name do aplicativo no Facebook Developer, onde terá que ser este mesmo, assim quem for contribuir e testar não precisa criar um app no face só para isto! Então a ideia é fornecer este ID do appface para facilitar os testes. Acho que seria interessante então apenas colocar no README os parametros do plugin! O que acha? |
Amigo, você utilizou alguma lib para fazer a autenticação?
Se sim, ela não está no package.json.
The text was updated successfully, but these errors were encountered: