Skip to content

Commit

Permalink
Update README.md
Browse files Browse the repository at this point in the history
  • Loading branch information
ReoHakase authored Jun 29, 2024
1 parent e0c87be commit 4a71877
Showing 1 changed file with 71 additions and 3 deletions.
74 changes: 71 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -84,6 +84,74 @@ pnpm turbo sb:test # Storybookのインタラクションテストを実行
pnpm turbo playwright:test # E2Eテストを実行
```

## その他情報

開発にかかった時間: 5日間 (1日あたり平均5時間作業)
## 評価

開発にかかった時間: 6日間 (1日あたり平均5時間作業)

> なお、担当者からのレビューは以下の通りです。
よろしければ参考までにご確認をいただけますと幸いです。

### Goodチェック
- 独自にワイヤーフレームを解釈してデザイン・実装している
- API で取得したデータをキャッシュしている
- API のエラーハンドリングができている
- 通信時のローディングの UI を実装している
- 良いインタラクションを与えるアニメーションを実装している
- Lighthouse のスコアで高い評価となっている
- コンポーネント分割の粒度が適切である
- ビジネスロジックをコンポーネントから外だししている
- CSS in JS を利用している
- Unit Test を書いている
- E2E Test を書いている
- StoryBook を導入している
- スナップショットテストが導入されている
- Docker を利用している
- CI を構築している
- Issue や PR の機能などを活用して開発している
- Hooks を使っている
- API-key をアプリケーションコードから秘匿している
- API-key をビルドファイルから秘匿している
- JSDoc を書いている
- コメントの量が適切である
- コミットの粒度が適切である
- コミットのメッセージが適切である
- コミットに prefix がついている
- Turborepo を使用している
- アイコンライブラリを使用している
- Zod を使用している
- Next 14 の App Router を使用している
- Storybook を活用した Interaction Test を実装している
- ダークモード対応をしている
- OGP 対応をしている
- favicon の設定をしている
- .npmrc の設定をしている
- LICENSE の設定をしている
- .prototools で Node.js と pnpm のバージョン指定をしている
- ワークスペースファイルを設定している
- Renovate を導入している
- Figma を活用している
- アクセシビリティ対応をしている
- トップページに人口比較のためのサンプルリンクを設置している
- Server Component を使用して API への fetch をサーバー側で実行している
- 実装の各所で多くの工夫が見られクオリティの高いアプリケーションになっている
- クエリパラメータによるビューの共有
- 独自のデザイン
- 適切なメタデータの設定
- etc…(書ききれない)
- README.md が詳細にかかれており、アピールポイントや技術スタックが非常にわかりやすい
- CI/CD を実装したうえでしっかりと全て通っている
- テストのクオリティが高い
- E2E テストでグラフが適切に表示されているか検証されている
- ユニットテストもしっかりとテストデータを用いて細かく検証されている
- セマンティックな HTML がかけている

### Nextチェック
- チェックボックス横のラベルにホバーした時に cursor: pointer になっていない
- トップページでチェックボックスをクリックしても表示が変わらない


### 総合評価
 Figma での UI デザイン設計に始まり、アプリケーションの設計はもちろん、実装意図を残したコメントや各テストの記述など、全てにおいて細部まで丁寧に実装されており、最善を目指して開発される姿勢がとても素晴らしいと思いました。極めて高レベルな対応を 30 時間という短い時間で成し遂げられており、完成した Web アプリケーションもクオリティの高いものになっており、レビュワー一同大変感心いたしました。チーム開発への意識もしっかりとされており、現時点で業務に加わっても第一線で活躍できる素養をお持ちの方だと確信いたしました。
 Next 項目もほとんどなかったですが、一点、ユーザー視点に立って Next を上げるとすれば、トップページで都道府県を選択するチェックボックスがあるものの、チェックを付けても何も表示が変わらないのはユーザーフレンドリーではないと感じました。ここを改善されるとより良い Web アプリケーションになるかと思います。

> レビュワー一同、非常に好評価でした。

0 comments on commit 4a71877

Please sign in to comment.