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

🐛 レシピの取得系APIの修正 & マイレシピの公開非公開フラグの適応 #137

Merged
merged 6 commits into from
Aug 12, 2023

Conversation

tuboihirokidesu
Copy link
Collaborator

@tuboihirokidesu tuboihirokidesu commented Aug 8, 2023

関連する issue*

Closes #133
Closes #134
Closes #135

作業内容*

メイン実装

以下2点の修正を行いました

  • 話題のレシピの取得 API が一般ユーザーのレシピも対象となっていたので、シェフのレシピに限定
  • 検索APIにて、検索ワードがない場合は話題のレシピの一覧取得。検索ワードがあれば、シェフの全レシピから検索をかける
2023-08-08.22.42.12.mov

また、レシピの公開・非公開のフラグをRecipeモデルに追加し、マイレシピ作成時はデフォルトで非公開にして、公開・非公開を切り替えられるように対応しました。

2023-08-09.18.56.53.mov

そのほか発生した実装

apiのprefixをactionsFor~,getRecipes~,getChefs~など管理しやすいように対応しました。

スクリーンショット 2023-08-09 18 58 39

またsession情報の呼び出しでcookieを呼び出していますが、現行の呼び出し方だとpnpm run buildコマンドでコケていたので、以下を参考にcookies の取得と使用の方法を定義し直しました

vercel/next.js#49373 (comment)

残してある課題

チェックリスト*

実装者

  • ターゲットブランチが適切に設定されている。
  • 新規でのバグ・警告等が残っていない。
  • 本 PR に関係のない差分を含んでいない。
  • PR の Assignee の設定。

その他

@tuboihirokidesu tuboihirokidesu self-assigned this Aug 8, 2023
@tuboihirokidesu tuboihirokidesu changed the title 🐛 レシピの取得系APIの修正 🐛 レシピの取得系APIの修正 & マイレシピの公開非公開フラグの適応 Aug 9, 2023
@tuboihirokidesu tuboihirokidesu merged commit c3eae4a into main Aug 12, 2023
@tuboihirokidesu tuboihirokidesu mentioned this pull request Aug 14, 2023
5 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
2 participants