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.

Sign up for GitHub

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

Jump to bottom

userPromptレイアウトの作成とapiリクエスト関数の生成 #100

Merged
merged 5 commits into from
Mar 29, 2024
Merged

userPromptレイアウトの作成とapiリクエスト関数の生成 #100

merged 5 commits into from
Mar 29, 2024

Conversation

dorimu0
Copy link
Contributor

@dorimu0 dorimu0 commented Mar 29, 2024

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

  • userPromptページのレイアウトを作成し、必要なapiリクエスト関数を生成しました。 そして、それに合わせたテストコードを作成しました。

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

  • userPromptページ生成によるレイアウト変更と機能実装です。

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

  • userPromptページに必要な画像を追加し、cssとinterfaceを追加しました。

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

  • userPromptページ内にエラーが発生しないか、apiリクエストが正常に動作しているか確認していただければ幸いです。

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

  • 新しく追加したapiリクエスト関数のテストコードを作成しました。

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

🖥 作動する様子

스크린샷 2024-03-29 오전 10 48 49

📌 PRを行う際の注意点

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

dorimu0 added 5 commits March 29, 2024 09:36
@dorimu0
✨ feat : Create user prompt layouts and pages
7e63351 
- Create a userContainer component to manage userPrompt components
- Material component Add later

Related issue: YJU-OKURA#92
@dorimu0
✨ feat : Create subComponents for userPrompt
b255f56 
- Manage each component with sub_containers
- Create sub_components for userPrompt
- Create interfaces for each props

Related issue: YJU-OKURA#92
@dorimu0
💄 style : Add userPrompt svgs and modify css
0496f79 
- Manage svgs with index files
- Define height using a CSS file

Related issue: YJU-OKURA#92
@dorimu0
✨ feat : Add api functions for subComponents in userPrompt
2229c52 
- Add promptChat, storage related API request functions
- Add an interface for the response value

Related issue: YJU-OKURA#92
@dorimu0
✅ test : Write API request function test code
e7d633d 
Test code for the sub_component's API request function

Related issue: YJU-OKURA#93
@dorimu0 dorimu0 added 🎨 Html&css 마크업 & 스타일링 ✅ Test test 관련(storybook, jest...) 📬 API 서버 API 통신 ✨ Feature 기능 개발 ⏰ P2 활발하게 고려해야 하는 이슈나 개선사항 labels Mar 29, 2024
@dorimu0 dorimu0 requested a review from a team March 29, 2024 01:49
@dorimu0 dorimu0 self-assigned this Mar 29, 2024
@Lainari
Copy link
Collaborator

Lainari commented Mar 29, 2024

P3 : 추후 Component는 Conventaion에 맞게 index.ts로 관리해주세요 :)

@Lainari Lainari merged commit e530152 into YJU-OKURA:main Mar 29, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cr-gpt cr-gpt[bot] cr-gpt[bot] left review comments

@yuminn-k yuminn-k yuminn-k approved these changes

Assignees

@dorimu0 dorimu0

Labels
📬 API 서버 API 통신 ✨ Feature 기능 개발 🎨 Html&css 마크업 & 스타일링 ⏰ P2 활발하게 고려해야 하는 이슈나 개선사항 ✅ Test test 관련(storybook, jest...)
Projects
ミノリ
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dorimu0 @Lainari @yuminn-k