AYU-Side-Hidden-Gems-Playlist / back

์ˆจ๋“ฃ๋ช…_๋ฐฑ์—”๋“œ
0 stars 0 forks source link

002-Playlist #11

Open didiha opened 6 months ago

didiha commented 6 months ago

๐Ÿ“ AS-IS (์ฝ”๋“œ ๋ฐ”๋€Œ๊ธฐ ์ „)

  1. post์˜ crud ๊ธฐ๋Šฅ ๊ตฌํ˜„ +) branch๋ช… post์—์„œ playlist๋กœ ๋ณ€๊ฒฝ

    • PostService์—์„œ PlaylistService๋กœ ์ด๋ฆ„ ๋ณ€๊ฒฝ
    • PlaylistService์— ๋…ธ๋ž˜ ์ถ”๊ฐ€/์ˆ˜์ •/์‚ญ์ œ ๊ธฐ๋Šฅ ์ถ”๊ฐ€
    • PostController์—์„œ PlaylistController๋กœ ์ด๋ฆ„ ๋ณ€๊ฒฝ
    • PlaylistController์— ๋…ธ๋ž˜ ์ถ”๊ฐ€/์ˆ˜์ •/์‚ญ์ œ ๊ธฐ๋Šฅ ์ถ”๊ฐ€
    • SongRepository ์ถ”๊ฐ€
    • SongForm ์ถ”๊ฐ€
    • Song ์ถ”๊ฐ€

โœจ To-Be (๋ฐ”๋€ ํ›„) ~ kakao ๋กœ๊ทธ์ธ ํ›„ ์ €์žฅ๋œ ๋ฉค๋ฒ„ ๋ฐ์ดํ„ฐ๊ฐ€ ํ•„์š”ํ–ˆ๊ธฐ ๋•Œ๋ฌธ์— Sign in ์ฝ”๋“œ๋ฅผ ๊ฐ€์ ธ์˜ด

~Service

~Controller

~Form

๐Ÿง ๋ฆฌ๋ทฐ์–ด๋“ค์ด ๋ดค์œผ๋ฉด ํ•˜๋Š” ์  [ ] ์—ฌ๋Ÿฌ ๋ฒˆ ์ˆ˜์ •์„ ํ•ด์„œ ์ฝ”๋“œ ์ค‘์— ๋น„ํšจ์œจ์ ์ธ ๋ถ€๋ถ„์ด ์žˆ์„ ์ˆ˜ ์žˆ์Šต๋‹ˆ๋‹ค. ์ด๋ฅผ ๋ฐœ๊ฒฌํ•ด์ฃผ์‹ ๋‹ค๋ฉด ๋ฐ”๋กœ ๋ฆฌ๋ทฐ ๋‹ฌ์•„์ฃผ์„ธ์š”! ์ˆ˜์ •ํ•˜๋„๋ก ํ•˜๊ฒ ์Šต๋‹ˆ๋‹ค. [ ] ์ฝ”๋“œ๋ฅผ ์ •๋ฆฌํ•œ๋‹ค๊ณ ๋Š” ํ–ˆ๋Š”๋ฐ ๋ถˆํ•„์š”ํ•œ ์ฝ”๋“œ๊ฐ€ ์žˆ๋‹ค๋ฉด ์ด๊ฒƒ๋„ ์–ธ๊ธ‰ํ•ด์ฃผ์‹œ๋ฉด ์‚ญ์ œํ•˜๋„๋ก ํ•˜๊ฒ ์Šต๋‹ˆ๋‹ค. [ ] member id๋กœ playlist ๋ชฉ๋ก์„ ์ฐพ์•„์˜ค๋Š” ๊ธฐ๋Šฅ์„ ์ถ”๊ฐ€ํ•œ ๊ฒƒ์ด๊ธฐ ๋•Œ๋ฌธ์— ์ด ๊ธฐ๋Šฅ์— ๋Œ€ํ•ด์„œ๋„ ๋ฆฌ๋ทฐ ๋‹ฌ์•„์ฃผ์‹œ๋ฉด ๊ฐ์‚ฌํ•˜๊ฒ ์Šต๋‹ˆ๋‹ค!

โœ”๏ธ TEST ์‚ฌํ•ญ [ o ] API TEST [ ] TEST CODE

DB๋Š” mysql, api๋Š” swagger๋กœ ํ…Œ์ŠคํŠธ๋ฅผ ์ง„ํ–‰ํ–ˆ์Šต๋‹ˆ๋‹ค.

๐Ÿ“ธ ํ…Œ์ŠคํŠธ ์Šคํฌ๋ฆฐ์ƒท ์Šคํฌ๋ฆฐ์ƒท 2024-04-01 081329

ํ”Œ๋ ˆ์ด๋ฆฌ์ŠคํŠธ ์ƒ์„ฑ

์Šคํฌ๋ฆฐ์ƒท 2024-04-01 081339

id๋Š” ์ž๋™์œผ๋กœ ์ฆ๊ฐ€๋˜๋ฉฐ ํ• ๋‹น๋˜๋Š” ๊ฑธ ํ™•์ธํ•  ์ˆ˜ ์žˆ์Œ.

์Šคํฌ๋ฆฐ์ƒท 2024-04-01 081352

DB์— ๋ฐ์ดํ„ฐ๊ฐ€ ์ž˜ ๋“ค์–ด์˜จ ๊ฑธ ํ™•์ธํ•  ์ˆ˜ ์žˆ์Šต๋‹ˆ๋‹ค.

์Šคํฌ๋ฆฐ์ƒท 2024-04-01 081426

ํ”Œ๋ ˆ์ด๋ฆฌ์ŠคํŠธ์— ๋…ธ๋ž˜ ์ถ”๊ฐ€

์Šคํฌ๋ฆฐ์ƒท 2024-04-01 081526

๋…ธ๋ž˜ ๋‘ ๊ฐœ๋ฅผ ์ถ”๊ฐ€ํ•œ ๋ชจ์Šต์ž…๋‹ˆ๋‹ค. ๋…ธ๋ž˜๋„ ์ž๋™์œผ๋กœ id๊ฐ€ ์ฆ๊ฐ€ํ•˜๋Š” ๋ชจ์Šต์„ ๋ณผ ์ˆ˜ ์žˆ์Šต๋‹ˆ๋‹ค.

์Šคํฌ๋ฆฐ์ƒท 2024-04-01 090420

๋…ธ๋ž˜ ์ •๋ณด ์ˆ˜์ •

์Šคํฌ๋ฆฐ์ƒท 2024-04-01 090428

์ˆ˜์ •์ด ๋œ ๋ชจ์Šต

์Šคํฌ๋ฆฐ์ƒท 2024-04-01 090504

์ˆ˜์ •๋œ ์ •๋ณด๊ฐ€ ์ž˜ ์ €์žฅ๋œ ๋ชจ์Šต

์Šคํฌ๋ฆฐ์ƒท 2024-04-01 090544

๋…ธ๋ž˜ ์‚ญ์ œ

์Šคํฌ๋ฆฐ์ƒท 2024-04-01 090550

๋…ธ๋ž˜ ์‚ญ์ œ ํ™•์ธ. DB ์‚ฌ์ง„์ด ์—†์ง€๋งŒ ์ž˜ ์‚ญ์ œ๋˜์—ˆ์Šต๋‹ˆ๋‹ค.


์—ฌ๊ธฐ์„œ๋ถ€ํ„ฐ๋Š” memberId ๋กœ playlist ์ฐพ๋Š” ๊ธฐ๋Šฅ ๋ถ€๊ฐ€ ์„ค๋ช…์ž…๋‹ˆ๋‹ค.

(((( ์ด๋ฏธ ๋กœ๊ทธ์ธ ํ•œ ์ƒํƒœ์ž„์„ ๊ฐ€์ •ํ•œ ๋’ค ์ž…๋‹ˆ๋‹ค. ))))

image

์šฐ์„  ์ด ์ •๋ณด๋ฅผ ๊ฐ€์ง„ ํšŒ์›์ด ๋กœ๊ทธ์ธ ๋˜์–ด member ์—”ํ‹ฐํ‹ฐ์— ์ €์žฅ๋œ ์ƒํƒœ์ž…๋‹ˆ๋‹ค.

image

์—ฌ๊ธฐ์„œ ์ด๋ฉ”์ผ์€ ํšŒ์›๊ฐ€์ž…๋œ ์‚ฌ์šฉ์ž์ธ์ง€ ํ™•์ธํ•˜๋Š” ๊ธฐ๋Šฅ๊ณผ ํ•ด๋‹น ํšŒ์›์˜ ์ •๋ณด๋ฅผ Playlist ๊ฐ์ฒด์— ์—ฐ๊ฒฐํ•˜๊ธฐ ์œ„ํ•ด ์‚ฌ์šฉ์ž์˜ ์ด๋ฉ”์ผ์„ ์‚ฌ์šฉํ•ฉ๋‹ˆ๋‹ค.

์Šคํฌ๋ฆฐ์ƒท 2024-05-02 163131

ํ”Œ๋ ˆ์ด๋ฆฌ์ŠคํŠธ๊ฐ€ ์ •์ƒ์ ์œผ๋กœ ๋งŒ๋“ค์–ด์ง€๋ฉด playlist ์—”ํ‹ฐํ‹ฐ์— playlistId์™€ member_FK๊ฐ€ ์ž˜ ์ €์žฅ๋˜์–ด ์žˆ๋Š” ๊ฑธ ํ™•์ธํ•  ์ˆ˜ ์žˆ์Šต๋‹ˆ๋‹ค.

image

ํ”Œ๋ ˆ์ด๋ฆฌ์ŠคํŠธ ์ƒ์„ฑ ํ›„ memberController์—์„œ memberId๋ฅผ ์ž…๋ ฅํ•˜๋ฉด

image

ํ•ด๋‹น memberId์— ์†ํ•ด ์žˆ๋Š” ํ”Œ๋ ˆ์ด๋ฆฌ์ŠคํŠธ ๋ชฉ๋ก์ด ์ถœ๋ ฅ ๋ฉ๋‹ˆ๋‹ค.

์ดํ›„ ํ”Œ๋ ˆ์ด๋ฆฌ์ŠคํŠธ ์ˆ˜์ •, ์‚ญ์ œ, ์ถ”๊ฐ€ ๋“ฑ๋“ฑ ๊ธฐ๋Šฅ์€ ๋™์ผํ•ฉ๋‹ˆ๋‹ค.

alswl0912 commented 6 months ago

https://docs.github.com/ko/pull-requests/collaborating-with-pull-requests/addressing-merge-conflicts/resolving-a-merge-conflict-on-github https://stackoverflow.com/questions/56683681/this-branch-has-conflicts-that-must-be-resolved-but-its-already-merged ๋จธ์ง€ ์‹œ ๋ณ‘ํ•ฉ ์ถฉ๋Œ ์˜ค๋ฅ˜์— ๊ด€ํ•œ ๋ธ”๋กœ๊ทธ ์ž…๋‹ˆ๋‹ค. ์ฐธ๊ณ ํ•ด๋ณด์„ธ์š”.