Confusing error message

NativeScript / nativescript-facebook

NativeScript plugin, wrapper of native Facebook SDK for Android and iOS

Apache License 2.0

79 stars 50 forks source link

Confusing error message #107

Closed Nestorferrando closed 6 years ago

Nestorferrando commented 6 years ago

https://github.com/NativeScript/nativescript-facebook/blob/2437e17b9aa71396a679335408c03011ef687bfa/src/login-manager.android.ts#L27

Hi! we noticed that any error here will be shown as "library not found", that can be confusing. For example, in our case the exception was because an invalid facebook id

maybe a console.log(e) would be a better option

DimitarTodorov commented 6 years ago

Hi @Nestorferrando, This issue is fixed with the latest version of nativescript-facebook plugin. https://github.com/NativeScript/nativescript-facebook/releases/tag/v2.2.3