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.

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

fix: listen to user doc changes instead of fetching doc to check if exists #226

Closed
wants to merge 4 commits into from
Closed

fix: listen to user doc changes instead of fetching doc to check if exists #226

wants to merge 4 commits into from

Commits on Oct 30, 2023

  1. fix(contribute command): listen to user doc changes instead of fetchi… 

    …ng doc to check if exists

There can be some lag after running the auth command before the user can be found in firebase. In
which case trying to fetch the user will fail with `FirebaseError: Unable to retrieve the
authenticated user.`. Instead we can listen and wait for user doc changes.

privacy-scaling-explorations#220
    @sripwoud
    sripwoud committed Oct 30, 2023
    Configuration menu
    Copy the full SHA
    4fd942f View commit details
    Browse the repository at this point in the history

  2. test(waitforuserdocumenttoexist): test new `waitForUserDocumentToExis… 

    …t` function
    @sripwoud
    sripwoud committed Oct 30, 2023
    Configuration menu
    Copy the full SHA
    a87f7b5 View commit details
    Browse the repository at this point in the history

  3. revert changes in contribute.test.ts

    @sripwoud
    sripwoud committed Oct 30, 2023
    Configuration menu
    Copy the full SHA
    8e238bd View commit details
    Browse the repository at this point in the history

Commits on Nov 2, 2023

  1. Merge branch 'dev' into sripwoud/issue-220

    @ctrlc03
    ctrlc03 committed Nov 2, 2023
    Configuration menu
    Copy the full SHA
    1b41157 View commit details
    Browse the repository at this point in the history