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

[FEAT] 대댓글 추가 연동 및 BottomSheet 수정 #86

Merged
merged 1 commit into from
Jan 19, 2024

Conversation

eshc123
Copy link
Collaborator

@eshc123 eshc123 commented Jan 19, 2024

Issue number and Link

이슈 번호 : #77

Summary

대댓글 추가 api 연동 및 Bottom Sheet 수정

*댓글 추가 시 에러가 나네요
{baseurl}/v1/comment/10/26 호출 시
"message":"사용자 상세 정보를 찾을 수 없습니다." 와 에러코드가 나와서
백엔드 개발자에게 문의해야할 것 같습니다.

PR Type

  • Feature
  • Bugfix
  • Code Style Update
  • Refactoring
  • Documentation content change
  • Other

Other Information

Common Type

- [UI] : UI 관련 작업
- [FEAT] : 새로운 기능 구현
- [ADD] : feat 이외의 부수적인 코드, 파일, 라이브러리 추가
- [MOD] : 코드 및 내부 파일 수정
- [CHORE] : 버전 코드, 패키지 구조, 함수 및 변수명 변경 등의 작은 작업
- [FIX] : 버그 및 오류 해결
- [DEL] : 불필요한 코드, 파일, 주석 삭제
- [DOCS] : READMEWiki 등의 문서 작업
- [REFACTOR] : 코드 리팩토링
- [MERGE] : 서로 다른 브랜치 간의 코드 병합
- [COMMENT] : 필요한 주석 추가 및 변경
- [SETTING] : 프로젝트 기초 세팅 관련 작업

branch

  • 모든 글자는 소문자로 작성한다.
feature/{type}-{작업 내용}

ex)
feature/feat-main-view
feature/add-font-res

@eshc123 eshc123 added ✨ feature New feature or request 석준 labels Jan 19, 2024
@eshc123 eshc123 requested review from arinming and Mnseo January 19, 2024 11:38
@eshc123 eshc123 self-assigned this Jan 19, 2024
Comment on lines +89 to +95
@POST("v1/comment/{noticeId}/{commentId}")
suspend fun addNoticeChildComment(
@Path("noticeId") noticeId: Int,
@Path("commentId") commentId: Int,
@Body commentContentRequestBody: CommentContentRequestBody,
): BaseResponse<CommentPayload>

Copy link
Owner

Choose a reason for hiding this comment

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

위에 남겨놓으신것처럼 스크린단이나 기능단 아니면 v1/v2로 분리할 필요성이 있어보여서
메모용으로 코멘트 남겨놓을게요 고생하셨습니다 ✔️🙇🏻‍♀️

@eshc123 eshc123 merged commit b28ca4e into master Jan 19, 2024
@eshc123 eshc123 deleted the feature/add-notice-re-comment branch January 28, 2024 03:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feature New feature or request 석준
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants