Feat : 즐겨찾기 페이지 API 구현 #54
Conversation
|
git merge develope 하고 다시 올려주신다는 거였죠? |
넵 develop 에서 다시 받아와서 push 했습니당 |
There was a problem hiding this comment.
와일드카드로 이미지 경로를 하면 전부 받아올 수 있으니 괜찮을 줄 알았는데
이미지 최적화 등 기능이 지원이 안된다고 하네요 ㅠ_ㅠ
추후에 리팩토링이 있다면 바꾸면 될거같아요
| <Image | ||
| src={info.imageSource || `/images/no-content.svg`} | ||
| objectFit="cover" | ||
| className="object-cover" |
components/LinkCard.tsx
Outdated
| } | ||
|
|
||
| const LinkCard = ({ isFavoritePage, ...info }: CardItemProps) => { | ||
| const [isSubscribed, seIsSubscribed] = useState(false); |
There was a problem hiding this comment.
앗 오타가 ..! se't'가 빠졌습니다
| // 즐겨찾기 폴더가 없는 경우 (404 처리) | ||
| if (err.response?.status === 404) { | ||
| return res.status(404).json({ message: "즐겨찾기 폴더가 없습니다." }); | ||
| } |
There was a problem hiding this comment.
스웨거에서 즐겨찾기 폴더 없을때 404 코드인건 확인했는데 어떤 상황에 즐겨찾기 폴더가 없는건지 잘 모르겠네욥
즐겨찾기한 링크가 없다는 걸까요?
There was a problem hiding this comment.
스웨거 문구에 즐겨 찾기 폴더가 없는 경우 라고 되어있긴한데, 조금 헷갈릴 수도 있을 것 같네요!
제 생각엔 즐겨찾기한 링크가 없을 때가 맞는 것 같습니당
| : null} | ||
| </CardsLayout> | ||
|
|
||
| {/* 즐겨찾기 항목이 없을 때 보여줄 메시지 (공통 컴포넌트로 사용할 건지 논의 필요) */} |
There was a problem hiding this comment.
별도의 컴포넌트로 뺀다면 해당 페이지 코드가 좀더 깔끔하긴 할것같습니다
|
|
||
| return ( | ||
| <div className="w-[340px] h-[344px] rounded-[12px] shadow-lg mt-20 ml-20 overflow-hidden cursor-pointer hover:scale-105 hover:duration-300"> | ||
| <div className="w-[340px] h-[344px] rounded-[12px] shadow-lg overflow-hidden cursor-pointer hover:scale-105 hover:duration-300"> |
There was a problem hiding this comment.
list.map 으로 가져오게 되니까 div대신 li 태그가 나을까요? 🙄
There was a problem hiding this comment.
li 로 사용하게되면 cardslayout 도 ul태그로 바뀌어야겟네욤 : ) 리팩토링 할 때 얘기해볼게여
이슈번호 #32
작업사항
로그인, 회원가입 페이지 tailwind 에 lg 추가했습니다.
LinkCard 컴포넌트에 props 추가 했습니다 ( 즐겨찾기에는 즐찾 아이콘과 케밥 버튼이 없어서 props 로 즐찾 페이지인지 true/false 받기)
즐찾 목록에 리스트가 없을 시에도 UI 구현했습니다.