n4mlz/sns-backend

sns-backend's Issues

短時間に同じ内容のポストを連投できないようにする

通知機能の実装

エラーハンドリング

Summary

• 大抵のエラーを 400 番で返してしまっている問題を解決したい

アカウント削除の実装

R2 に保存している画像のファイル名を UUID にする

Summary

• ファイル名が更新されないとファイルの更新は CDN でのキャッシュの更新依存になってしまう

改行を含むポストができない

Description

• 改行を含む文章はスペースに置き換えられてしまう

CORS の設定

Summary

• CORS を設定する

• backend

• Cloudflare R2

ポストの詳細画面でのバグ

Description

• ポストの詳細表示画面にて、コメントや返信に相互フォローでないユーザーがいるとポストそのものが表示できなくなる

API の作成

Summary

• API を作成する

---

API

• POST /api/posts
• GET /api/posts/{postId}
• DELETE /api/posts/{postId}
• POST /api/posts/comments
• DELETE /api/posts/comments/{commentId}
• POST /api/posts/replies
• DELETE /api/posts/replies/{replyId}
• GET /api/posts/{postId}/likes
• PUT /api/posts/like
• PUT /api/posts/unlike
• GET /api/posts/timeline
• GET /api/users/{userName}
• GET /api/users/{userName}/posts
• GET /api/users/{userName}/mutuals
• PUT /api/follows/follow
• PUT /api/follows/unfollow
• GET /api/follows/requests
• PUT /api/follows/reject
• PUT /api/settings/profile
• PUT /api/settings/profile/icon
• PUT /api/settings/profile/bgImage

以下は Cloudflare R2 で代用

• GET /api/users/{userName}/icon

• GET /api/users/{userName}/bg-image

• (いずれは /api/notification も欲しい)

ポストの詳細ページでの通知の挙動

Description

• ポストの詳細ページに飛んだ時、未読のコメントや返信が含まれていたらそのコメントや返信の通知を確認済みにする

ユーザー名変更の API をプロフィール変更の API から分離する

Description

• プロフィール画像を変更しつつユーザー名を変更するときに困る

ドメインモデルの見直し

Summary

• User エンティティ一つ一つに UserRepository を持たせるのは間違いではないか？
• UserFactory に UserRepository を持たせるのは正しいのか？
  • これが正しくない場合、NewUserFactory() や SetDefaultUserFactory() はいらなくなるのではないか？
• UserId のみを必要とする操作のときに、毎回 getUser() をして DB アクセスが発生するのは非効率なのではないか？
  • 構造体の埋め込みを利用して、Userid のみを必要とする操作と User 自体を必要とする操作で構造体を分けられないか

相互フォローの相互フォローを覗けないようにする

ポスト・コメント・返信の削除

Description

• Foreign Key として繋がっているものがあると削除出来ない (ex. いいねされたポストは削除できないなど)

ユーザー名更新時に R2 上で既に同じ名前がないか検証していない

環境変数の設定

Summary

• DB の接続設定などを docker や gorm から参照するときに、環境変数を使うようにする

N+1 問題の解決

Summary

• ネストされたクエリを修正する
• N+1 問題が発生する部分はインフラ層に分離されている

ページネーションに対応

Summary

• 現状、一覧取得をする API はページネーションに対応していないので対応したい

ユーザー名の変更をプロフィール変更の API から分離する

Description

• プロフィール画像の更新とプロフィールの更新を同時にしたいときに困る