[Feat] 難易度（絶対評価）を投票できるようにしましょう

[KATO-Hiro]

[river0525]

@KATO-Hiro
AtCoder IDを用いた認証機能の有効化に関してお尋ねしたいことがございます。

/src/routes/users/edit/+page.svelteの84行目あたりに
<!-- FIXME: フォーム送信時に、基本情報ページが表示されてしまう不具合が修正されるまでは非公開 -->
と書かれたコメントを発見したのですが、具体的にはどのような不具合が起こっていたでしょうか。

[KATO-Hiro]

@river0525

教えていただき、ありがとうございます。

これはユーザが認証フォームから登録情報を送信すると、画面のリロードが発生してしまう実装であるということです。
このため認証用のタブではなく基本情報のタブに遷移してしまい、UX の観点から望ましくない状況です。

アカウント連携自体は、実装当時はできていたと記憶していますが、認証のサーバを 開発メンバーのあべみさん個人のものから、NoviSteps アカウントのものに移行する必要があります。

念のため確認ですが、投票機能の本体の実装がほぼ程度完了しており、認証機能の改修に取り組んでいるという認識でいいのでしょうか?

[KATO-Hiro]

上記に関連して、相談させていただきたいことがあります。

今回の機能追加に伴い、以下のサブタスクも可能な限り実施したいです。
初期の調査・実装工数は掛かるかもしれませんが、プロジェクト全体の機能追加・修正を円滑に進めるために必要だと考えています。

DBのモデルの分割の方向性や、既存のソースコードのリファクタリングの是非についてご意見・ご感想をいただけますでしょうか?
あくまでも叩き台なので、優先順位付けの結果、後回しや対処しないと判断することもあるかと思います。

関連タスク

• DBのテーブルを単一の責務となるようにモデルの分割
  • User モデルから AtCoderアカウントに関する属性を別モデルとして切り出し
  • 上記に関連する該当コードを特定して修正
• src/lib や src/routes/users/edit にある アカウント関連のプロダクションコード、テストコードをsrc/features/account に集約
• 認証用サーバをNoviSteps アカウントに移行

// 現状
model User {
  id                        String   @id @unique
  // here you can add custom fields for your user
  // e.g. name, email, username, roles, etc.
  username                  String   @unique
  role                      Roles    @default(USER)
  atcoder_validation_code   String   @default("")
  atcoder_username          String   @default("")
  atcoder__status Boolean? @default(false)
  created_at                DateTime @default(now())
  updated_at                DateTime @updatedAt

  auth_session Session[]
  key          Key[]
  taskAnswer   TaskAnswer[]
  workBooks    WorkBook[]

  @@map("user")
}


// 修正（イメージ）
model User {
// 追加（1対1のリレーションが必要であれば追加）
atCoderAccount AtCoderAccount

// 削除
// atcoder_validation_code, atcoder_username, atcoder__status

// 上記以外はそのまま
}

model AtCoderAccount {
  id 
  handle                      String   @default("") // 旧atcoder_username
  isValidated               Boolean? @default(false)
  validationCode        String   @default("")
  createdAt                 DateTime @default(now())
  updatedAt                DateTime @updatedAt
}

[river0525]

@KATO-Hiro
ご返信ありがとうございます。

念のため確認ですが、投票機能の本体の実装がほぼ程度完了しており、認証機能の改修に取り組んでいるという認識でいいのでしょうか?

本体の実装はDBのみ完了しており、ロジックの方は認証機能を有効化してから作成する方針でした。
先に認証抜きで実装を進める方向に切り替えます。

DBのモデルの分割の方向性や、既存のソースコードのリファクタリングの是非についてご意見・ご感想をいただけますでしょうか?

その件に関しましては私も賛成です。
現状実装が完了していない箇所が多いため、認証機能の有効化以外が完成したタイミングで再度お話しできたらと思います。