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の追加とレイアウトの生成 #110

Merged
merged 7 commits into from
Apr 21, 2024

Conversation

Lainari
Copy link
Collaborator

@Lainari Lainari commented Apr 21, 2024

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

  • クラス選択、メインページで実際のデータベースに反映されるAPI接続を行いました。
  • 関連するレイアウトを追加しました。
  • 一部のスタイルを変更しました。

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

  • クラス作成と照会APIの接続が完了しました。
  • クラス投稿&スケジュールの作成、削除、照会APIの接続が完了しました。

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

  • Navbarのポジションスタイルが変更されました
  • レイアウトが追加され、それに伴うinterfaceファイルが生成されました。

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

  • 実際にクラスを生成して調べてみてください。
  • まだ、投稿&スケジュール修正APIは問題解決中なので、解決されたらすぐに反映します。

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

image
image

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

🖥 作動する様子

  • なし

📌 PRを行う際の注意点

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

Lainari added 7 commits April 21, 2024 16:15
- Unify card components that maintain the same style
- Manage many props at once with interface files

Related Issue : YJU-OKURA#59
- Repositioning to eliminate overlap with other components

Related Issue : YJU-OKURA#71
- All Class Lookup API
- Class lookup API by role
- Class creation API

Related Issue : YJU-OKURA#109
- Change the path from _group to _class

Related Issue : YJU-OKURA#78
- Create Class Schedule API
- Delete Class Schedule API
- Get Class Schedule API
- Adding layout and the react-calendar library

Related Issue : YJU-OKURA#101 YJU-OKURA#99
- Create Post API
- Delete Post API
- Get post API
- Add interface files for props

Related Issue : YJU-OKURA#101
…props

- Creating a class and changing the layout structure for an entry
- Adding zIndex props to the Dashboard

Related Issue : YJU-OKURA#77
@Lainari Lainari added 🎨 Html&css 마크업 & 스타일링 📬 API 서버 API 통신 ✨ Feature 기능 개발 ⏰ P1 반드시 해결해야 하는 중요한 이슈나 문제 (예: 버그, 취약점 등) labels Apr 21, 2024
@Lainari Lainari requested a review from a team April 21, 2024 08:03
@Lainari Lainari self-assigned this Apr 21, 2024
.eslintrc.json Show resolved Hide resolved
next.config.js Show resolved Hide resolved
package.json Show resolved Hide resolved
@Lainari
Copy link
Collaborator Author

Lainari commented Apr 21, 2024

P2 : 수정 API 연결 문제 해결 한 후에 추가 PR 하겠습니다

@Lainari Lainari merged commit 3a68cf4 into YJU-OKURA:main Apr 21, 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 통신 ✨ Feature 기능 개발 🎨 Html&css 마크업 & 스타일링 ⏰ P1 반드시 해결해야 하는 중요한 이슈나 문제 (예: 버그, 취약점 등)
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants