Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[#112][FEATURE] DailyNote API 연결 #116

Open
wants to merge 3 commits into
base: dev-release
Choose a base branch
from
Open

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.

2 participants