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

시니또 정보 조회 api 기능 수정 #135

Merged
merged 6 commits into from
Nov 4, 2024
Merged

시니또 정보 조회 api 기능 수정 #135

merged 6 commits into from
Nov 4, 2024

Conversation

Dobbymin
Copy link
Contributor

#️⃣ 연관된 이슈

ex) #이슈번호, - #이슈번호

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요.(이미지 첨부 가능)

  • api 수정에 따라 reponse type 수정
  • 로그인 , 회원가입시 isSinitto 속성 localStorage에 추가

스크린샷 (선택)

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

⏰ 현재 버그

  • 마이페이지도 수정할려니까 계좌조회랑 시니또 정보조회랑 뭔가 엉켰는지 오류가 발생하는것 같네요..
  • 마이페이지 기능 수정하고 이거 머지 눌러주세용 @Diwoni

✏ Git Close

close #이슈번호

@Dobbymin Dobbymin added 😡 Bug Fix 발생한 버그를 수정하는 경우 ✨ Feature 새로운 기능 추가 및 구현하는 경우 labels Oct 31, 2024
@Dobbymin Dobbymin requested review from Diwoni and JYN523 October 31, 2024 12:10
@Dobbymin Dobbymin self-assigned this Oct 31, 2024
@Dobbymin Dobbymin linked an issue Oct 31, 2024 that may be closed by this pull request
Copy link
Collaborator

@JYN523 JYN523 left a comment

Choose a reason for hiding this comment

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

깔끔하게 잘 작성하셨네요!
카톡을 방금 확인해서 마이페이지 수정 후에 오류 해결하고 머지하면 될 것 같습니다.

고생하셨어요!

@Diwoni Diwoni merged commit 6dbe971 into Weekly Nov 4, 2024
1 check passed
@Diwoni Diwoni deleted the Feat/issue-#133 branch November 4, 2024 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
😡 Bug Fix 발생한 버그를 수정하는 경우 ✨ Feature 새로운 기능 추가 및 구현하는 경우
Projects
None yet
Development

Successfully merging this pull request may close these issues.

시니또 메인 페이지 기능 수정
3 participants