Skip to content

Conversation

@hongggyelim
Copy link
Collaborator

작업 내용


- 링크페이지에서 바로 사용하실 수도 있을것같아 해당 PR 먼저 올립니다.
- 이어서 폴더 수정, 삭제 등 모달 기능 구현 하겠습니다
- EditModal은 작업중이라 무시해주셔도 좋습니다

@hongggyelim hongggyelim self-assigned this Nov 9, 2024
@hongggyelim hongggyelim linked an issue Nov 9, 2024 that may be closed by this pull request
@junjeeong junjeeong self-requested a review November 9, 2024 02:51
Copy link
Collaborator

@99minji 99minji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다-!

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FolderItem li 태그에 css 로 cursor-pointer 추가해주면 좋을 것 같아요!

image

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

폴더에 추가는 중복으로도 가능한건가요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • 커서 포인터 추가하겠습니다!
  • 스웨거 보니까 링크 추가 시 하나의 폴더 id 를 지정해야하네요! 지금은 단순히 ui만 있는거라 중복 선택이 안되도록 추후 반영하겠습니다

Copy link
Contributor

@mungyun mungyun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Zustand도 잘 사용해주셨고 전반적으로 깔끔한 것 같네요! 고생하셨습니다 🙂‍↕️

Comment on lines +1 to +20
import axiosInstance from "@/lib/api/axiosInstanceApi";
import axios from "axios";
import { NextApiRequest, NextApiResponse } from "next";

const handler = async (req: NextApiRequest, res: NextApiResponse) => {
const token = req.cookies.accessToken;
console.log("Token:", token); // 쿠키 확인

if (!token) {
return res.status(401).json({ error: "사용자 정보를 찾을 수 없습니다." });
}

if (req.method === "POST") {
try {
const response = await axiosInstance.post("/folders", req.body, {
headers: {
Authorization: `Bearer ${token}`,
},
});
console.log(response);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

깔끔하게 잘 구현하셨군요 👍

@hongggyelim hongggyelim merged commit 28be0f6 into develop Nov 9, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

모달 기능 구현 - 폴더 생성 POST api

4 participants