各ページの作成

[ZOI-dayo]

デフォルトのページを削除した後、次のようにルーティングを設定しました。

パス	内容
/	トップ
/signup	登録 (メールでの登録・SNSアカウントでの登録を選べる)
/signup/register	新規登録用フォーム (メールアドレスによる登録の際、ユーザー名やパスワードを設定する)
/signup/after-mail	「確認メールを送信しました」の画面
/login	ログイン画面 (メールでのログイン・SNSアカウントでのログインを選べる)
/reset-password	パスワードリセットのためメールアドレスを打ち込む画面
/reset-password/form	新しいパスワードを設定する画面
/reset-password/after-mail	「確認メールを送信しました」の画面
/users/:id	ユーザーのプロフィールページ
/users/:id/submissions	ユーザーの提出一覧ページ
/users/:id/problems	そのユーザーが作成した問題の一覧ページ
/problems	サイト上にあるすべての問題の一覧ページ
/problems/:id	問題文が書いてあるページ
/problems/:id/submit	コードを回答するためのページ
/problems/:id/submissions	その問題に対する提出の一覧ページ
/problems/:id/submissions/:submissionId	特定の提出の詳細なステータスやコードを閲覧するためのページ
/problems/:id/editional	問題の解説が書いてあるページ
/problems/:id/edit	管理者のみが閲覧できる、問題や解説・テストケースなどを編集するためのページ
/settings/	(→ /settings/accountにリダイレクト)
/settings/account	アカウント情報 (ユーザー名やメールアドレス、アカウント連携など)を編集するページ
/settings/profile	自己紹介を記入するページ

@YuHima03 の意見を聞きたいです (コードのレビューに関してはその後)

[YuHima03]

ありがとう!
とりあえずざっと見たので簡単にコメントします.

• 解説は1つの問題に対して複数用意できるはずなので, /problems/:id/editorials の方が良いかも. (ただ, 他のサイトを見ると editorial になっているので単数形で良いのかも......?)
• ↑と同様の理由で, 個別の解説ページ/problems/:id/editorials/:editorialId と解説の編集ページ /problems/:id/editorials/:editorialId/edit が要ります.
• 「確認メールを送信しました」のページを用意するかどうかはちょっと決まってないので保留でお願いします.
• 問題の編集とジャッジの編集は分ける方針ですがまだ確定ではないのでこれも保留でお願いします.

詳しい話は追ってコメントします.

[ZOI-dayo]

他のページ(users、problems、settings)が複数形になっているので、editorialsとする方が良いと思いました
/problems/:id/editorialsへの変更、/problems/:id/editorials/:editorialsalIdと/problems/:id/editorials/:editiorialId/editの追加、だけ先にやっておきました

[ZOI-dayo]

ページの実装タスクをメンバーに割り振るなら、このPRがマージされてからのほうが、実装すべきファイルがわかりやすく、他ページへのリンクなども貼りやすいのでタスクを振りやすいと考えています。
この観点だけで言えば、例えば/signup/after-mailは/signup/からしかリンクが貼られないと思うので、マージしてしまったあとに、仕様が明確に定まってから/signup/の実装担当者に適宜修正してもらうという形でもそこまで問題はないのかな、と思っています。
(まだ実装を振る段階になっていない気がするので、今すぐにこのPRをマージしよう! という主張ではないです)

[mathsuky]

ありがとうございます！このようなPRを先にマージすると作業しやすいという意見に全面的に賛成なので，これを先にマージする方向で進めていきたいと思います。

[mathsuky]

一点質問なのですが，例えばsignup関連のページでSignupView.vueの中に色々なviewを入れている意図はありますか？
個人的には，まだデザインが固まっておらずどれぐらいの共通部分があるかもわからないので，現状では各画面ごとに単一のViewを切っても良いのではないのかと思います。また，現状のような形式にするとしても，Viewというファイルの中にViewというファイルが入るのは少し不思議な命名な気がします。

[ZOI-dayo]

users、problems、settingsについては、サイドメニューで画面切り替えをすると思いこの形にしています。サイドメニューをProblemViewなどの親に配置し、各タブの内容をProblem**Viewに書くイメージです。
signup、reset-passwordについては、単にURLの親子関係から上のページ達と同じ構造にしただけで特に強い理由はありません(すみません)。確定している共通要素もそこまでなさそうなので、確かに分離してしまったほうが良さそうです。

入れ子となっている子要素の名前については、確かにViewだと不自然なので、末尾のViewを外してしまおうかと思っています。
名前の先頭が親Viewのものなので、他の一般のコンポーネントとは混ざらないはずです。

[mathsuky]

users、problems、settingsについては、サイドメニューで画面切り替えをすると思いこの形にしています。サイドメニューをProblemViewなどの親に配置し、各タブの内容をProblem**Viewに書くイメージです。

そっか，サイドバーはApp.vueに書けば良いと思っていたが，画面によって要素が変わるんですよね。それであれば現状の構成がよさそうなので，こちらは命名の変更だけお願いします:pray:

signup、reset-passwordについては、単にURLの親子関係から上のページ達と同じ構造にしただけで特に強い理由はありません(すみません)。確定している共通要素もそこまでなさそうなので、確かに分離してしまったほうが良さそうです。

こちらについては分離して再度出していただければと思います。今後必要になれば各ページの担当者(間)で形式を変更してもらいましょう。

よろしくお願いいたします。