Skip to content

Conversation

@choiyoorim
Copy link
Contributor

✅ PR check list

  • PR 제목: "[#이슈번호] 작업 내용"
  • commit message가 적절한지 확인해주세요.
  • 적절한 branch로 요청했는지 확인해주세요.
  • Assignees, Label을 붙여주세요.
  • 주의 사항과 관련해 꼭 확인해야 할 사람이 있다면 Reviewer로 등록해주세요.

📝 PR 요약

  • DailyNote GET API 통해서 데이터 가져오기
  • DailyNote POST API 통해서 데이터 보내기
  • 헤더에 임시 토큰 값 사용

📌 변경 사항 및 주의 사항

일단 날짜 부분은 고쳐지고 난 다음에 바꿔두도록 하겠습니다
이전과 달리 POST 진행시 각각 Post하도록 바꾸었는데 그이유는 일단 API 보내는 데이터 구조가 바뀌면서
Body 각각을 지정해주어야 하기 때문에 구조를 바꾸었습니다.
더 좋은 방법이 있는지는 고민중...


@choiyoorim choiyoorim added ✨ feature 새로운 기능을 개발해요. 유림 유림이가 했어요! labels Mar 12, 2023
@choiyoorim choiyoorim self-assigned this Mar 12, 2023
Copy link
Member

@huiseon37 huiseon37 left a comment

Choose a reason for hiding this comment

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

고생해쑈~

Comment on lines 62 to 73
export const postEmojiData = async (data: EmojiRequestType) => {
const post = await client.post('/dailynote', { ...data });
return post;
};

export const postFeelData = async (data: FeelRequestType) => {
const post = await client.post('/dailynote', { ...data });
return post;
};

export const postMemoData = async (data: MemoRequestType) => {
const post = await client.post('/dailynote', { ...data });
Copy link
Member

Choose a reason for hiding this comment

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

요거 합칠수 있는 방법이 있을건데 한번 내부분 끝나고 나도 손대볼게~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ feature 새로운 기능을 개발해요. 유림 유림이가 했어요!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants