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接続、ライブチャット機能を追加 #169

Merged
merged 3 commits into from
Jun 19, 2024

Conversation

Lainari
Copy link
Collaborator

@Lainari Lainari commented Jun 19, 2024

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

  • クラス参加やメンバー管理、ライブチャット機能を追加しました。

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

  • クラス参加モーダルとメンバー管理ページ、画像共有ページに機能を追加しました

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

  • 機能を効果的に見ることができるようにレイアウトの変更が行われました

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

  • クラスへの参加やビデオ授業でのチャット機能をテストしてほしいです

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

  • なし

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

🖥 作動する様子

  • なし

📌 PRを行う際の注意点

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

Lainari added 3 commits June 19, 2024 21:08
- Required for live chat functionality

Related issue: YJU-OKURA#166
- Joining a Class
- Connect to the Member Management API
- Verify class code and password

Related Issue : YJU-OKURA#109
- Chat icon in the video classroom space
- Multi-user enabled confirmed

Related Issue : YJU-OKURA#167
@Lainari Lainari added 🎨 Html&css 마크업 & 스타일링 🔨 Refactor 코드 리팩토링 📬 API 서버 API 통신 ✨ Feature 기능 개발 ⏰ P1 반드시 해결해야 하는 중요한 이슈나 문제 (예: 버그, 취약점 등) labels Jun 19, 2024
@Lainari Lainari requested a review from a team June 19, 2024 13:36
@Lainari Lainari self-assigned this Jun 19, 2024
Copy link
Contributor

@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.

ソクゲイ~

@Regulus0811 Regulus0811 merged commit db75e95 into YJU-OKURA:main Jun 19, 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 반드시 해결해야 하는 중요한 이슈나 문제 (예: 버그, 취약점 등) 🔨 Refactor 코드 리팩토링
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants