Feat/recent add #88
Feat/recent add #88
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Pull Request Overview
์ต์ ๊ณก ๊ธฐ๋ฅ์ ์ถ๊ฐํ๊ณ , ์ฝ๋ ์ ๋ฐ์ ๊ฑธ์ณ ์ผ๊ด์ฑ์ ๊ฐ์ ํ PR์ ๋๋ค. ์๋ก์ด ์ต์ ๊ณก ์กฐํ ํ์ด์ง๋ฅผ ๋ง๋ค๊ณ , API ์๋ํฌ์ธํธ ๋ค์ด๋ฐ์ snake_case์์ kebab-case๋ก ํต์ผํ์ต๋๋ค.
- ์๋ณ๋ก ์ถ๊ฐ๋ 100๊ฐ ๊ณก์ ์กฐํํ ์ ์๋ ์๋ก์ด '/recent' ํ์ด์ง ์ถ๊ฐ
- API ์๋ํฌ์ธํธ ๋ค์ด๋ฐ ๊ท์น์ kebab-case๋ก ํต์ผ (snake_case โ kebab-case)
- ํ๋จ ๋ค๋น๊ฒ์ด์ ๋ฉ๋ด ํญ๋ชฉ ๋ณ๊ฒฝ ๋ฐ ์ฌ์ดํธ๋งต ์กฐ์
Reviewed Changes
Copilot reviewed 26 out of 42 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| apps/web/src/app/recent/page.tsx | ์๋ณ ์ต์ ๊ณก ์กฐํ ๊ธฐ๋ฅ์ ์ ๊ณตํ๋ ์ ํ์ด์ง ์ถ๊ฐ |
| apps/web/src/app/recent/RecentSongCard.tsx | ์ต์ ๊ณก ๋ชฉ๋ก์ ๊ฐ๋ณ ์์ดํ ์ ํ์ํ๋ ์ปดํฌ๋ํธ |
| apps/web/src/lib/api/recentAddSong.ts | ์๋ณ ์ต์ ๊ณก ๋ฐ์ดํฐ๋ฅผ ๊ฐ์ ธ์ค๋ API ํด๋ผ์ด์ธํธ ํจ์ |
| apps/web/src/queries/recentAddSongQuery.ts | ์ต์ ๊ณก ๋ฐ์ดํฐ๋ฅผ ์ํ React Query ํ |
| apps/web/src/app/api/songs/recent-add/route.ts | ์๋ณ ์ต์ ๊ณก ๋ฐ์ดํฐ๋ฅผ ์ ๊ณตํ๋ ๋ฐฑ์๋ API ์๋ํฌ์ธํธ |
| apps/web/src/lib/api/* | API ์๋ํฌ์ธํธ๋ฅผ kebab-case๋ก ํต์ผ (user_stats โ user-stats ๋ฑ) |
| apps/web/src/queries/* | ์ฟผ๋ฆฌ ํค ๋ค์ด๋ฐ์ kebab-case๋ก ๋ณ๊ฒฝ |
| apps/web/src/Footer.tsx | ํ๋จ ๋ค๋น๊ฒ์ด์ ์ '์ต์ ๊ณก' ๋ฉ๋ด ์ถ๊ฐ ๋ฐ ๋ ์ด์์ ๊ฐ์ |
apps/web/src/app/recent/page.tsx
Outdated
| setToday(new Date(today.getFullYear(), today.getMonth() + 1, 1)); | ||
| setPrevAction('next'); | ||
| }; | ||
| console.log('recentAddSongs', recentAddSongs, today.getFullYear(), today.getMonth() + 1); |
There was a problem hiding this comment.
ํ๋ก๋์ ์ฝ๋์์ ๋๋ฒ๊น ์ฉ console.log๋ฅผ ์ ๊ฑฐํด์ผ ํฉ๋๋ค. ๊ฐ๋ฐ ๊ณผ์ ์์ ์ฌ์ฉํ๋ ์ฝ๋๋ก ๋ณด์ด๋ ์ญ์ ํด ์ฃผ์ธ์.
| console.log('recentAddSongs', recentAddSongs, today.getFullYear(), today.getMonth() + 1); |
| 'use client'; | ||
|
|
||
| import StaticLoading from '@/components/StaticLoading'; | ||
| // import StaticLoading from '@/components/StaticLoading'; |
There was a problem hiding this comment.
์ฃผ์ ์ฒ๋ฆฌ๋ import ๋ฌธ๋ค์ ์ ๊ฑฐํด์ผ ํฉ๋๋ค. ์ฌ์ฉํ์ง ์๋ ์ฝ๋๋ ์ฝ๋๋ฒ ์ด์ค๋ฅผ ๊น๋ํ๊ฒ ์ ์งํ๊ธฐ ์ํด ์ญ์ ํ๋ ๊ฒ์ด ์ข์ต๋๋ค.
| // import StaticLoading from '@/components/StaticLoading'; |
| import { useRecentSongQuery } from '@/queries/recentSongQuery'; | ||
| import { useSaveSongFolderQuery } from '@/queries/saveSongFolderQuery'; | ||
| import { useRecentSingLogQuery } from '@/queries/recentSingLogQuery'; | ||
| // import { useSaveSongFolderQuery } from '@/queries/saveSongFolderQuery'; |
There was a problem hiding this comment.
์ฃผ์ ์ฒ๋ฆฌ๋ import ๋ฌธ๋ค์ ์ ๊ฑฐํด์ผ ํฉ๋๋ค. ์ฌ์ฉํ์ง ์๋ ์ฝ๋๋ ์ฝ๋๋ฒ ์ด์ค๋ฅผ ๊น๋ํ๊ฒ ์ ์งํ๊ธฐ ์ํด ์ญ์ ํ๋ ๊ฒ์ด ์ข์ต๋๋ค.
| // import { useSaveSongFolderQuery } from '@/queries/saveSongFolderQuery'; |
| import { Check, ChevronsDown, ChevronsUp, GripVertical, Trash } from 'lucide-react'; | ||
|
|
||
| import { Button } from '@/components/ui/button'; | ||
| import { Card } from '@/components/ui/card'; |
There was a problem hiding this comment.
์ฌ์ฉํ์ง ์๋ import ๋ฌธ์ ์ ๊ฑฐํ์ง๋ง, Tooltip ๊ด๋ จ ์ฝ๋๊ฐ ์ค์ ๋ก ์ฌ์ฉ๋์ง ์๋์ง ํ์ธํด์ผ ํฉ๋๋ค. ๋ง์ฝ ๋ค๋ฅธ ๊ณณ์์ ์ฌ์ฉ๋๋ค๋ฉด ํด๋น ๊ธฐ๋ฅ์ด ์ ์ ์๋ํ๋์ง ํ ์คํธ๊ฐ ํ์ํฉ๋๋ค.
๐ PR ์ ๋ชฉ
Feat : ์ต์ ๊ณก ๊ธฐ๋ฅ ์ถ๊ฐ
๐ ๋ณ๊ฒฝ ์ฌํญ
๐ฌ ์ถ๊ฐ ์ฐธ๊ณ ์ฌํญ