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

Add re-auth method #554

Merged
merged 3 commits into from
Apr 3, 2022
Merged

Add re-auth method #554

merged 3 commits into from
Apr 3, 2022

Conversation

bannzai
Copy link
Owner

@bannzai bannzai commented Apr 2, 2022

Abstract

アカウント連携部分で、セルを長押しすると再連携できる隠し機能を追加

Why

DBとFirebaseのAuthentification上では、ユーザーIDにGoogleアカウントが紐づいているのに、google/isLinked が falseが帰ってきてしまう場合がある。これはFirebaseから帰ってくる情報なのでブラックボックスでなぜこの結果になるのかがよくわかってない。ユーザーに再度「連携」を試してもらうと「すでに登録済みのユーザー」と返ってくる。

追加で連携はできないが再連携(認証情報の更新)をしたらうまくいくかも。と思ったのでとりあえず隠し機能的に追加。リリース次第ユーザーに案内する

Links

Checked

  • Analyticsのログを入れたか
  • 境界値に対してのUnitTestを書いた
  • パターン分岐が発生するWidgetに対してWidgetTestを書いた
  • リリースノートを追加した

@bannzai bannzai merged commit 2820f5b into main Apr 3, 2022
@bannzai bannzai deleted the add/function/re-auth branch April 3, 2022 03:39
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.

1 participant