Merge pull request #1273 from AtCoder-NoviSteps/#1271

🎨 Update descriptions and figures (1271)

Author: KATO-Hiro

README.md

@@ -36,7 +36,7 @@
 <div align="center">
   <video
     width="720"
-    src="https://github.com/user-attachments/assets/88b19b21-fda3-4337-8266-9243b89e7096"
+    src="https://github.com/user-attachments/assets/5d714055-e2fa-47e7-aa50-6e34a977fa58"
     autoplay
     muted
     controls
@@ -49,13 +49,13 @@
 
 - **回答状況を自分で記録できる**: 「AC (正解)」「解説AC」「挑戦中」「未挑戦」から選べます。
 - **一歩先の問題に挑戦**: 17段階で難易度付けされており、自分の実力に合った問題が探せます。
-- **(開発中) 問題集で得意を伸ばす・苦手を克服**: 例題・類題を通して、各トピックの基礎から応用的な方法まで身につけられます。
+- **問題集で得意を伸ばす・苦手を克服**: 例題・類題を通して、各トピックの基礎から応用的な方法まで身につけられます。
 
 ## 主な機能
 
 - 問題の回答状況の記録
 - 問題一覧: 難易度(ヒューリスティックな判断を含む)を見て、問題を探す
-- (開発中) 問題集: 特定のトピックに関する例題・類題を集中して解く
+- 問題集(アルファ版): 特定のトピックに関する例題・類題を集中して解く
 
 ## 予備知識
 
@@ -71,7 +71,7 @@
 
 1. (初回のみ) ユーザ名とパスワードを入力して、アカウントを作成します。
 2. ログインします。お試し用のアカウント (ユーザ名: guest、パスワード: Hell0Guest) を利用することもできます。
-3. 問題一覧で、グレードおよび問題を選び、回答欄の「更新」をクリックもしくはタップします。
+3. 問題集や問題一覧で、グレードおよび問題を選び、回答欄の「更新」をクリックもしくはタップします。
 4. 該当する回答状況 (デフォルトは「未挑戦」)を選択し、「回答を更新」ボタンを押します。
 
 ## 本サービスのスポンサー(敬称略・順不同)
@@ -102,7 +102,7 @@
 
 ## ご要望、ご報告、お気付きの点など
 
-- [連絡先](https://twitter.com/acnovisteps)
+- [連絡先](https://x.com/acnovisteps)
 
 - [本プロジェクトへの参加方法](https://github.com/KATO-Hiro/AtCoderNovisteps/blob/main/CONTRIBUTING.md)
 
@@ -121,7 +121,7 @@
   - 貢献内容(予定も含む): アプリの仕様に関する提案・技術選定・設計・実装
 - [@ウルズニャー](https://x.com/uruzunyaa)
   - 役割: 問題の難易度評価
-  - 貢献内容(予定も含む): 問題の難易度評価
+  - 貢献内容(予定も含む): 問題の難易度評価、問題集作成
 - [@nonon](https://x.com/nonon_math)
   - 役割: 問題の難易度評価
   - 貢献内容(予定も含む): 問題の難易度評価、問題集作成

src/lib/components/AuthForm.svelte

@@ -14,8 +14,8 @@
     GUEST_USER_NAME,
     GUEST_USER_PASSWORD,
     // GUEST_USER_PASSWORD_FOR_LOCAL,
-    LOGIN_LABEL,
   } from '$lib/constants/forms';
+  import { HOME_PAGE, LOGIN_PAGE } from '$lib/constants/navbar-links';
 
   // FIXME: 構造体に相当するものを利用した方が拡張・修正がしやすくなるかもしれせまん
   export let formProperties;
@@ -55,7 +55,7 @@
         throw new Error('Not found form element or HTMLFormElement');
       }
 
-      const response = await fetch('/login', {
+      const response = await fetch(LOGIN_PAGE, {
         method: 'POST',
         body: new FormData(formElement),
       });
@@ -65,7 +65,7 @@
         return;
       }
 
-      await goto('/');
+      await goto(HOME_PAGE);
     } catch (error) {
       console.error('Failed to login as a guest: ', error);
     } finally {
@@ -88,21 +88,20 @@
       <h3 class="text-xl font-medium text-gray-900 dark:text-white">{title}</h3>
       <MessageHelperWrapper message={$message} />
 
-      {#if title === LOGIN_LABEL}
-        <Button
-          type="button"
-          class="w-full"
-          on:click={handleLoginAsGuest}
-          on:keydown={handleLoginAsGuest}
-          disabled={isSubmitting || $submitting}
-        >
-          <div class="text-md">お試し用のアカウントでログイン</div>
-        </Button>
+      <!-- Login as guest -->
+      <Button
+        type="button"
+        class="w-full"
+        on:click={handleLoginAsGuest}
+        on:keydown={handleLoginAsGuest}
+        disabled={isSubmitting || $submitting}
+      >
+        <div class="text-md">お試し用のアカウントでログイン</div>
+      </Button>
 
-        <div>
-          <Hr classHr="my-2 h-0.5 bg-gray-400 dark:bg-gray-200" />
-        </div>
-      {/if}
+      <div>
+        <Hr classHr="my-2 h-0.5 bg-gray-400 dark:bg-gray-200" />
+      </div>
 
       <!-- User name -->
       <div class="space-y-2">

src/lib/constants/http-response-status-codes.ts

@@ -3,6 +3,7 @@
 
 // Redirection messages
 export const FOUND = 302;
+export const SEE_OTHER = 303;
 export const TEMPORARY_REDIRECT = 307;
 
 // Client error responses

src/lib/constants/navbar-links.ts

@@ -1,14 +1,24 @@
+export const HOME_PAGE = `/`;
+export const ABOUT_PAGE = `/about`;
+export const SIGNUP_PAGE = `/signup`;
+export const LOGIN_PAGE = `/login`;
+export const WORKBOOKS_PAGE = `/workbooks`;
+export const PROBLEMS_PAGE = `/problems`;
+
+// For Admin
+export const IMPORTING_PROBLEMS_PAGE = `/tasks`;
+export const TAGS_PAGE = `/tags`;
+
 export const navbarLinks = [
-  { title: `ホーム`, path: `/` },
-  { title: `問題一覧`, path: `/problems` },
-  // FIXME: 問題集機能を一般公開するときにコメントを外す。
-  // { title: `問題集（アルファ版）`, path: `/workbooks` },
-  { title: `サービスの説明`, path: `/about` },
+  { title: `ホーム`, path: HOME_PAGE },
+  { title: `問題集（アルファ版）`, path: WORKBOOKS_PAGE },
+  { title: `問題一覧`, path: PROBLEMS_PAGE },
+  { title: `サービスの説明`, path: ABOUT_PAGE },
 ];
 
 export const navbarDashboardLinks = [
-  { title: `問題のインポート`, path: `/tasks` },
-  { title: `問題一覧`, path: `/problems` },
-  { title: `問題集`, path: `/workbooks` },
-  { title: `タグ一覧`, path: `/tags` },
+  { title: `問題のインポート`, path: IMPORTING_PROBLEMS_PAGE },
+  { title: `問題一覧`, path: PROBLEMS_PAGE },
+  { title: `問題集`, path: WORKBOOKS_PAGE },
+  { title: `タグ一覧`, path: TAGS_PAGE },
 ];

src/routes/(auth)/login/+page.server.ts

@@ -11,17 +11,18 @@ import { auth } from '$lib/server/auth';
 
 import {
   BAD_REQUEST,
-  FOUND,
+  SEE_OTHER,
   INTERNAL_SERVER_ERROR,
 } from '$lib/constants/http-response-status-codes';
+import { HOME_PAGE } from '$lib/constants/navbar-links';
 
 import type { Actions, PageServerLoad } from './$types';
 
 export const load: PageServerLoad = async ({ locals }) => {
   const session = await locals.auth.validate();
 
   if (session) {
-    redirect(FOUND, '/');
+    redirect(SEE_OTHER, HOME_PAGE);
   }
 
   const form = await superValidate(null, zod(authSchema));
@@ -82,6 +83,6 @@ export const actions: Actions = {
 
     // redirect to
     // make sure you don't throw inside a try/catch block!
-    redirect(FOUND, '/problems');
+    redirect(SEE_OTHER, HOME_PAGE);
   },
 };

src/routes/(auth)/logout/+page.server.ts

@@ -3,24 +3,26 @@
 import { auth } from '$lib/server/auth';
 import { fail, redirect } from '@sveltejs/kit';
 
+import { SEE_OTHER, UNAUTHORIZED } from '$lib/constants/http-response-status-codes';
+import { HOME_PAGE } from '$lib/constants/navbar-links';
+
 import type { Actions } from './$types';
 
 export const load = () => {
-  redirect(303, '/');
+  redirect(SEE_OTHER, HOME_PAGE);
 };
 
-// TODO: ステータスコードを定数で書き換え
 export const actions: Actions = {
   logout: async ({ locals }) => {
     const session = await locals.auth.validate();
 
     if (!session) {
-      return fail(401);
+      return fail(UNAUTHORIZED);
     }
 
     await auth.invalidateSession(session.sessionId); // invalidate session
     locals.auth.setSession(null); // remove cookie
 
-    redirect(303, '/');
+    redirect(SEE_OTHER, HOME_PAGE);
   },
 };

src/routes/(auth)/signup/+page.server.ts

@@ -10,13 +10,20 @@ import { LuciaError } from 'lucia';
 import { authSchema } from '$lib/zod/schema';
 import { auth } from '$lib/server/auth';
 
+import {
+  SEE_OTHER,
+  BAD_REQUEST,
+  INTERNAL_SERVER_ERROR,
+} from '$lib/constants/http-response-status-codes';
+import { HOME_PAGE } from '$lib/constants/navbar-links';
+
 import type { Actions, PageServerLoad } from './$types';
 
 export const load: PageServerLoad = async ({ locals }) => {
   const session = await locals.auth.validate();
 
   if (session) {
-    redirect(302, '/');
+    redirect(SEE_OTHER, HOME_PAGE);
   }
 
   const form = await superValidate(null, zod(authSchema));
@@ -30,7 +37,7 @@ export const actions: Actions = {
     const form = await superValidate(request, zod(authSchema));
 
     if (!form.valid) {
-      return fail(400, {
+      return fail(BAD_REQUEST, {
         form: {
           ...form,
           message:
@@ -68,7 +75,7 @@ export const actions: Actions = {
         (e instanceof PrismaClientKnownRequestError && e.code === 'P2002') ||
         (e instanceof LuciaError && e.message === 'AUTH_DUPLICATE_KEY_ID')
       ) {
-        return fail(400, {
+        return fail(BAD_REQUEST, {
           form: {
             ...form,
             message:
@@ -77,7 +84,7 @@ export const actions: Actions = {
         });
       }
 
-      return fail(500, {
+      return fail(INTERNAL_SERVER_ERROR, {
         form: {
           ...form,
           message: 'サーバでエラーが発生しました。本サービスの開発・運営チームに連絡してください。',
@@ -87,6 +94,6 @@ export const actions: Actions = {
 
     // redirect to
     // make sure you don't throw inside a try/catch block!
-    redirect(302, '/');
+    redirect(SEE_OTHER, HOME_PAGE);
   },
 };