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

コーディングテスト機能 #7935

Merged
merged 13 commits into from
Jan 14, 2025
Merged

コーディングテスト機能 #7935

merged 13 commits into from
Jan 14, 2025

Conversation

komagata
Copy link
Member

@komagata komagata commented Jul 6, 2024

管理画面 コーディングテスト一覧(/mentor/coding_tests)

image

管理画面 コーディングテスト作成 (/mentor/coding_tests/new)

image

プラクティス詳細(のコーディングテスト一覧)(/practices/1234

image

コーディングテスト一覧(/practices/1234/coding_tests

image

コーディングテスト詳細(/coding_tests/1234)

image

コーディングテスト提出物一覧(/coding_tests/1234/coding_test_submissions

image

コーディングテスト提出物詳細(/coding_tests/1234/coding_test_submissions/1234

image

@komagata komagata self-assigned this Jul 6, 2024
@komagata komagata force-pushed the spike/programming-problem branch from aa43c92 to b90f3a4 Compare July 28, 2024 09:14
@komagata komagata force-pushed the spike/programming-problem branch 2 times, most recently from d98b5fd to ad717e6 Compare August 12, 2024 08:06
@komagata komagata changed the title プログラミングテスト機能 コーディングテスト機能 Aug 12, 2024
@komagata komagata force-pushed the spike/programming-problem branch from 5b9588b to e50a00f Compare August 12, 2024 18:29
@komagata
Copy link
Member Author

komagata commented Aug 12, 2024

@machida まだUI変わるかもなのでしっかりテストを書いてない感じですが、お時間ある時にデザインお願いします。(トップページ優先で大丈夫です)

@komagata
Copy link
Member Author

komagata commented Aug 12, 2024

📝 URL構成やUIなどとりあえずで本決まりではないので、気になるところがあったらご相談いただければ幸いです〜。

@machida
Copy link
Member

machida commented Aug 30, 2024

📝
TODO#7935

@machida machida force-pushed the spike/programming-problem branch from 7138e94 to 712ac48 Compare August 30, 2024 07:57
@machida machida force-pushed the spike/programming-problem branch from 68df3ce to 518ed2f Compare October 15, 2024 02:46
@machida
Copy link
Member

machida commented Oct 15, 2024

@komagata
rebaseしましたー
コーディングテストのtabの部分だけ、view_component に合わせて実装をお願いします:pray:

@komagata komagata force-pushed the spike/programming-problem branch from 518ed2f to 6bb3dc3 Compare October 15, 2024 05:40
@komagata
Copy link
Member Author

@machida rebaseありがとうございました〜!

@komagata
Copy link
Member Author

@machida 下記のpill navの部分なんですが、一つのコーディングテスト(最初の出力)に対して「自分の回答」は1つですが、「みんなの回答」は複数個あるので、分類としてわかりにくくならないか気になったのですがいかがでしょうか。

また、このpill navは、「コーディングテストの提出物一覧」と「コーディングテストの提出物個別」の両方に表示されていますが、これはあっていますでしょうか?

スクリーンショット 2024-10-16 6 14 15

お手数おかけしますが確認いただければありがたいです〜

@machida
Copy link
Member

machida commented Oct 15, 2024

@komagata


指摘ありがとうございます!
pill nav を消して、代わりに

「自分の提出 show」←→「回答 index 」or 「回答 show」のナビゲーションを
タイトル横に置くことにしました。

貼り付けた画像_2024_10_16_7_57 貼り付けた画像_2024_10_16_7_57

リンクは設定済です。


TODO#7935

で検索すると、対応が必要な箇所がでるようになっています。
残りの作業はコーディングテスト関係のページにタブの設置する作業になります。
タブの設置をお願いします。

タブは、コーディングテストタブをpractice_page_tabsに追加し、コーディングテスト関係のページはコーディングテストタブがactiveになるようお願いします。

@komagata
Copy link
Member Author

@machida

「自分の提出 show」←→「回答 index 」or 「回答 show」のナビゲーションを
タイトル横に置くことにしました。

なるほどです、ありがとうございます〜!

TODO#7935

で検索すると、対応が必要な箇所がでるようになっています。
残りの作業はコーディングテスト関係のページにタブの設置する作業になります。

かしこまりました。検索しやすくて助かります〜!

@komagata komagata force-pushed the spike/programming-problem branch from c13be99 to fa42bc0 Compare November 20, 2024 05:03
@komagata komagata force-pushed the spike/programming-problem branch from 33de13e to acfe554 Compare January 9, 2025 16:29
test_caseが無くても登録可能だったため。
@komagata komagata force-pushed the spike/programming-problem branch from 11496e9 to 5a55850 Compare January 13, 2025 19:24
@komagata komagata merged commit 3f8808d into main Jan 14, 2025
1 check passed
@komagata komagata deleted the spike/programming-problem branch January 14, 2025 01:17
@github-actions github-actions bot mentioned this pull request Jan 14, 2025
32 tasks
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.

2 participants