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

グループボードのCRUD機能の実装 #48

Merged
merged 5 commits into from
Feb 26, 2024

Conversation

yuminn-k
Copy link
Member

🔍 このPRで解決したい問題は何ですか?

  • グループボード機能の追加とAPIの拡張です。このPRは、グループボードに対するCRUD操作(作成、読み取り、更新、削除)を実装し、APIの機能範囲を拡大します。これにより、ユーザーがグループボードをより効果的に管理できるようになります。

✨ このPRで主に変わったことは何ですか?

  • グループボードのCRUD操作のための新しいコントローラーメソッドを追加しました。
  • グループボードデータを扱うためのサービスロジックとリポジトリメソッドを実装しました。
  • グループボードのためのDTO(データ転送オブジェクト)を作成しました。

🔖 主な変更点以外に追加で変更された部分はありますか?

  • なし

🙏🏻 Reviewerに特に見ていただきたい部分はありますか?

  • 新しく追加されたグループボードのサービスロジックとエンドポイントの処理フローに特に注意してレビューしていただきたいです。

🩺 このPRでテストや検証が必要な部分はありますか?

  • 新しいエンドポイントの動作確認。
  • CRUD操作のテスト。

📚 関連するIssueやTrello、ドキュメント

🖥 作動する様子

(ここにスクリーンショットやビデオ、またはgifを追加)

📌 PRを行う際の注意点

  • Reviewerはコードレビュー時に良いコードの方向性を示しますが、コード修正を強制することはありません。
  • Reviewerは良いコードを見つけた場合、賞賛と励ましを惜しみません。
  • レビューは特別なケースでない限り、Reviewerに指定された時点から3日以内に行ってください。
  • コメント作成時にPrefixにP1、P2、P3を書いていただくと、Assigneeがより明確にコメントに対して対応することができます。
    • P1 : 必ず反映してください (Request Changes) - 問題が発生したり、脆弱性が発見されたケースなど。
    • P2 : 反映を積極的に検討していただければと思います (コメント)。
    • P3 : こんな方法もあるんじゃないかな~などの些細な意見です (Chore)。

Updated Swagger documentation to include new routes and models for group board management in the API. This includes the addition of group board create, update, delete, and list endpoints with detailed descriptions and necessary parameters.

Related issue:
* YJU-OKURA#42
* YJU-OKURA#9
Implemented full CRUD (Create, Read, Update, Delete) operations for group board functionality in the API. This includes new controller methods, service logic, DTOs (Data Transfer Objects), and repository methods to handle group board data.

Related issue: YJU-OKURA#42
Refactored the main function and router setup for better readability and maintainability. Extracted controller initialization and route setup into separate functions. Adjusted the way environment variables and migrations are handled.

Related issue: YJU-OKURA#42
Enhanced the group board feature by integrating image upload functionality. Modified the group board service and controller to handle image uploads using AWS S3.

Related issue: YJU-OKURA#42
Updated and added new constants for error and success messages. This standardizes responses across different parts of the application, ensuring consistency in user-facing messages.

Related issue: YJU-OKURA#42
@yuminn-k yuminn-k added ⏰ P1 반드시 해결해야 하는 중요한 이슈나 문제 (예: 버그, 취약점 등) ✨ Feature 기능 개발 🔨 Refactor 코드 리팩토링 📬 API 서버 API 통신 📃 Docs 문서 작성 및 수정 (README.md 등) 🆗 safe CI workflow 동작 관련 labels Feb 26, 2024
Copy link
Collaborator

@Regulus0811 Regulus0811 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다👍

Copy link
Member

@Z00One Z00One left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

お疲れ!!

@yuminn-k yuminn-k removed the 🆗 safe CI workflow 동작 관련 label Feb 26, 2024
@yuminn-k yuminn-k merged commit fcd8834 into YJU-OKURA:main Feb 26, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📬 API 서버 API 통신 📃 Docs 문서 작성 및 수정 (README.md 등) ✨ Feature 기능 개발 ⏰ P1 반드시 해결해야 하는 중요한 이슈나 문제 (예: 버그, 취약점 등) 🔨 Refactor 코드 리팩토링
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants