NativeScript / NativeScript/firebase

[auth][android] signOut return undefined

Open
#174 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
TypeScript
Stars
62
Forks
53
Avg merge
8d 4h
Merged PRs (30d)
2

Description

Hello @NathanWalker and @triniwiz

The signOut on android return undefined instead of boolean! the workaround is fixed by adding async await but this will force us to change the return type to Promise! and if we did we should doing same for IOS as well.

What do you think ?

https://github.com/NativeScript/firebase/blob/889726640381cb4a0cdbe855f658ff3245a0eb3d/packages/firebase-auth/index.android.ts#L1398

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with packages/firebase-auth/index.android.ts around line 1398, where the issue reports the Android signOut return value. Compare the Android behavior with the iOS implementation and confirm the intended boolean versus Promise contract; done means the platform APIs return the agreed value consistently.

Written by the indexing model from the issue text.

Assessment

Tech stack
android, typescript
Domain
authentication, mobile
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.