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

FE-54๐Ÿ”€ epic๋ธŒ๋žœ์น˜ ์ตœ์‹ ํ™” #90

Merged
merged 14 commits into from
Jul 28, 2024

Conversation

jisurk
Copy link
Contributor

@jisurk jisurk commented Jul 28, 2024

๐Ÿ”– Issue Ticket

์—†์Œ

โœ๏ธ Description

epic/FE-54--detail-page ๋ธŒ๋žœ์น˜ ์ตœ์‹ ํ™”

โœ… Checklist

PR

  • Branch Convention ํ™•์ธ

epic/ ์—ํ”ฝ, feat/ ํ”ผ์ณ, fix/ ๋ฒ„๊ทธ ์ˆ˜์ •, refactor/ ๊ฐœ์„ 

  • Base Branch ํ™•์ธ
  • ์ ์ ˆํ•œ Label ์ง€์ •
  • Assignee ๋ฐ Reviewer ์ง€์ •

Test

  • ๋กœ์ปฌ ์ž‘๋™ ํ™•์ธ

Additional Notes

  • (์—†์Œ)

newjinlee and others added 14 commits July 15, 2024 10:21
๋‹จ์ˆœ ๋ž˜ํ•‘ ๊ธฐ๋Šฅ๋ฐ–์— ์—†๋Š” ์ปดํฌ๋„ŒํŠธ ์‚ญ์ œ / emotionselector์—์„œ emotioniconcard๋ฅผ ์ง์ ‘ ์‚ฌ์šฉํ•˜๋„๋ก ์ˆ˜์ •
FE-61 ๐Ÿ”จ ๊ณต์šฉ ์ปดํฌ๋„ŒํŠธ ์ˆ˜์ •
FE-46 ๐Ÿ”จ ๊ณต์šฉ ์ปดํฌ๋„ŒํŠธ ๋ฉ˜ํ† ๋ง ์ˆ˜์ •์‚ฌํ•ญ ๋ฐ˜์˜
* โž• ์ด๋ฏธ์ง€ ํŒŒ์ผ ์ถ”๊ฐ€

* ๐Ÿ’„ ๋กœ๊ทธ์ธ ํŽ˜์ด์ง€ ๋ ˆ์ด์•„์›ƒ ์ƒ์„ฑ

* ๐Ÿ’„ ๋กœ๊ทธ์ธ ํŽ˜์ด์ง€ UI ์ƒ์„ฑ ๋ฐ ๋ฐ˜์‘ํ˜• ๋””์ž์ธ ๊ตฌํ˜„

* FE-60 โœจ react hook form, zod ์ถ”๊ฐ€

* FE-60 ๐Ÿ’„ ๋กœ๊ทธ์ธ ํผ ์Šคํƒ€์ผ ์ˆ˜์ •

- ํ…์ŠคํŠธ ์ธํ’‹ ํ…Œ๋‘๋ฆฌ
- ๋กœ๊ทธ์ธ ๋ฒ„ํŠผ

* FE-60 โ™ป๏ธ ๋กœ๊ทธ์ธ ์Šคํ‚ค๋งˆ ๋ถ„๋ฆฌ

* โœจ ๋กœ๊ทธ์ธ ์‘๋‹ต ๋ฐ์ดํ„ฐ ์Šคํ‚ค๋งˆ ์ •์˜

* โœจ ๋กœ๊ทธ์ธ api ์ƒ์„ฑ

* โœจ ์š”์ฒญ๊ณผ ์‘๋‹ต์— ๊ด€ํ•œ ์ธํ„ฐ์…‰ํ„ฐ ์ถ”๊ฐ€

* โœจ useSignin mutation hook ์ƒ์„ฑ

* โšก useSignin hook ๋กœ๊ทธ์ธ ํผ์— ์ ์šฉ

* ๐Ÿ”ฅ AuthLayout ์‚ญ์ œ

* ๐ŸŽจ onSubmit ํ•จ์ˆ˜ ์ธ๋ผ์ธ์œผ๋กœ ์ •์˜

* โ™ป๏ธ ์‘๋‹ต ์ธํ„ฐ์…‰ํ„ฐ์˜ ์—๋Ÿฌ ์ฒ˜๋ฆฌ ๋ฐ ํ† ํฐ ๊ฐฑ์‹  ๋กœ์ง ๊ฐœ์„ 

* โ™ป๏ธ postSignin api ์—๋Ÿฌ์ฒ˜๋ฆฌ ๋กœ์ง ์‚ญ์ œ

* ๐Ÿ”ฅ useSignin hook ์‚ญ์ œ

* ๐Ÿšš useSigninMutation hook์œผ๋กœ ์ด๋ฆ„ ๋ณ€๊ฒฝ ๋ฐ ํŒŒ์ผ ์ด๋™

* โœจ Toaster ์ปดํฌ๋„ŒํŠธ ์ถ”๊ฐ€

* โœจ toast๋กœ ์—๋Ÿฌ๋ฉ”์‹œ์ง€ ๋„์šฐ๊ธฐ
* FE-64๐Ÿ’„ ๊ธ€์ž‘์„ฑ ํŽ˜์ด์ง€ UI์ถ”๊ฐ€ (#44)

* FE-72 โœจ ์—ํ”ผ๊ทธ๋žจ ๋“ฑ๋ก api์—ฐ๋™ (#52)

* FE-72โœจ ๊ธ€์ž‘์„ฑํŽ˜์ด์ง€ ์Šคํ‚ค๋งˆ ์ถ”๊ฐ€

* FE-72โœจ formํƒœ๊ทธ Form์ปดํฌ๋„ŒํŠธ๋กœ ๋ณ€๊ฒฝ

* FE-72โœจ ํƒœ๊ทธ ์ €์žฅ๊ธฐ๋Šฅ ์ถ”๊ฐ€

* FE-72โœจ ์—ํ”ผ๊ทธ๋žจ ๋“ฑ๋ก api์—ฐ๋™

* FE-72โœจ ์—ํ”ผ๊ทธ๋žจ ๋“ฑ๋ก์‹œ ํ•ด๋‹น ์—ํ”ผ๊ทธ๋žจ ํŽ˜์ด์ง€๋กœ ์ด๋™ ๊ธฐ๋Šฅ ์ถ”๊ฐ€

* FE-72โœจ ๋“ฑ๋ก ์ค‘์ผ๋•Œ์˜ ๋กœ์ง์ถ”๊ฐ€

* FE-72โœจ  toast-> alert-dailog๋กœ ๋ณ€๊ฒฝ

* FE-72๐Ÿ“ TODO์ฃผ์„ ์ถ”๊ฐ€

---------

Co-authored-by: ์šฐ์ง€์„ <[email protected]>

* FE-73โœจ ์œ ํšจ์„ฑ๊ฒ€์‚ฌ ์ถ”๊ฐ€ (#66)

* FE-73โ™ป๏ธ  Tag๊ด€๋ฆฌ ํ•จ์ˆ˜ ํ›…์œผ๋กœ ๋ถ„๋ฆฌ

* FE-73โœจ  RadioGroup ๋กœ์ง ์ˆ˜์ •

* FE-73โœจ ์œ ํšจ์„ฑ๊ฒ€์‚ฌ ์ถ”๊ฐ€

* FE-73โ™ป๏ธ  ์ €์ž ๋ณธ์ธ ์„ ํƒ์‹œ์˜  ๋กœ์ง ๋ณ€๊ฒฝ

* FE-73โœจ ์ค‘๋ณต ํƒœ๊ทธ ๊ฒ€์‚ฌ ๋กœ์ง ์ถ”๊ฐ€

* FE-73โ™ป๏ธ ์ถœ์ฒ˜ ์œ ํšจ์„ฑ(optional)๊ฒ€์‚ฌ ์ˆ˜์ •

* FE-73โœจ  ํ•„์ˆ˜ํ•ญ๋ชฉ ์ž…๋ ฅํ–ˆ์„๋•Œ ๋ฒ„ํŠผ ํ™œ์„ฑํ™”

* FE-73๐Ÿ› ํƒœ๊ทธ๋ฅผ ์ž…๋ ฅํ–ˆ๋‹ค๊ฐ€ ์ง€์› ์„๋•Œ ๋ฒ„ํŠผ ํ™œ์„ฑํ™”๋˜์žˆ๋Š” ๋ฒ„๊ทธ ์ˆ˜์ •

* FE-73๐Ÿ› useEffect ์˜์กด์„ฑ๋ฐฐ์—ด lint problem ํ•ด๊ฒฐ

* FE-73๐Ÿ› url์œ ํšจ์„ฑ๊ฒ€์‚ฌ ์—๋Ÿฌ ๋ฉ”์„ธ์ง€ ์•ˆ๋œจ๋Š” ๋ฒ„๊ทธ ์ˆ˜์ •

---------

Co-authored-by: ์šฐ์ง€์„ <[email protected]>

* FE-71โ™ป๏ธ epic๋ธŒ๋žœ์น˜ ์ฝ”๋“œ๋ฆฌ๋ทฐ ๋ฐ˜์˜ (#76)

* FE-71โ™ป๏ธ  token,interceptor ๋กœ์ง ์ˆ˜์ •

* FE-71โ™ป๏ธ  AddEpigram ์ฝ”๋“œ๋ฆฌ๋ทฐ ๋ฐ˜์˜

* FE-71๐Ÿ”ฅ ํ…Œ์ŠคํŠธ์šฉ ์ƒ์„ธํŽ˜์ด์ง€ ์‚ญ์ œ

* FE-71โ™ป๏ธ  onKeyDown -> onKeyUp ์ˆ˜์ •

---------

Co-authored-by: ์šฐ์ง€์„ <[email protected]>
@jisurk jisurk added Priority: High ๐Ÿ”ฅ ์šฐ์„ ์ˆœ์œ„ ๋†’์Œ Type: Merge ๐Ÿšš ๋จธ์ง€ labels Jul 28, 2024
@jisurk jisurk requested a review from a team July 28, 2024 02:53
@jisurk jisurk self-assigned this Jul 28, 2024
@jisurk jisurk merged commit 16ede2c into epic/FE-54--detail-page Jul 28, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: High ๐Ÿ”ฅ ์šฐ์„ ์ˆœ์œ„ ๋†’์Œ Type: Merge ๐Ÿšš ๋จธ์ง€
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants