wafflestudio20-5 / team3-server Goto Github PK

View Code? Open in Web Editor NEW

5.0 1.0 2.0 318 KB

Kotlin 99.94% Dockerfile 0.06%

team3-server's Introduction

🥕 Waffle-Market 🥕

🥕 당근마켓 클론 코딩 와플마켓 API 서버

당근마켓 기능을 클론 코딩한 와플마켓 API 서버입니다.
단순 게시판 기능 외에 예약, 거래, 채팅 등의 부가 기능이 더해진 서비스를 구현해보면서 구성원들의 개발 실력 성장을 이루고자 했고, 서비스 기획에 인풋을 줄이기 위해 기존에 있는 서비스를 기반으로 클론 코딩을 하였습니다.

🎞️ 와플마켓 서비스 소개 영상 및 사진

🥕 와플마켓 서비스 소개 영상 링크 🥕

👨‍👩‍👧‍👦 누가 만들었나요?

김좌훈(JwahoonKim)	김준형(leeeryboy)	곽민지(Joanne19-drive)

🏛️ 서버 구조

📚 기술 스택

🧑‍💻Convention

프로젝트를 시작 전 아래와 같은 컨벤션을 정함으로써 코드 관리에 효율성을 높였습니다.

👔 Code Convention

팀원 간 코드 스타일을 맞추고 가독성을 높여 코드 리뷰를 원할하게 하기 위해 ktlint를 사용하였습니다.
제일 많이 쓰이는 jlleitschuh/ktlint-gradle을 사용하였습니다.
Github Action 을 이용하여 main 브랜치에 PR 올릴때마다 자동으로 스타일 검사를 하도록 설정하였습니다.

😎 Commit Convention

아래와 같은 커밋 컨벤션을 통해 서로의 작업물이 어떤 유형의 것인지 파악하기 쉽도록 하였습니다.

⛙ Git Branch 전략 및 Merge 전략

Github-flow를 사용했습니다.
- 크지 않은 사이즈의 토이프로젝트였기 때문에 간단한 Branching 전략인 Github-flow를 사용했습니다.
- 각자의 작업물은 feature 브랜치를 만들어 관리했으며, 완성한 작업물은 main 브랜치에 Squash Merge 전략을 사용해 합쳤습니다.
아래와 같은 근거로 Merge 전략은 Squash Merge 전략을 택했습니다.
- 각자 작업한 결과물의 완성본이 main 브랜치의 관심사였던 점
- PR을 날리는 단위를 작게 하자고 미리 협의한 점
- 각자 작업물의 히스토리는 main 브랜치에 남기기보다는 로컬에 남겨 관리하는 것이 편하겠다고 판단한 점

📖 기술적인 이슈 및 개발 과정을 겪으며 정리한 글

DB Diagram

🎉 서로에게 남기는 롤링페이퍼

🧑‍💻 준형이에게

우리팀 막내이면서 팀장을 맡아준 준형이ㅋㅋㅋ 어려운거 뚝딱뚝딱 다 잘해줘서 고맙고 든든했당.
기회가 되면 또 같이 작업할 수 있었으면 좋겠고 담학기 수업 많이 겹쳐서 재밌겠다 같이 열심히 들어보자! 블로그도 열심히써라!
본인이 왜 팀장인지 모르겠다고 했지만 앞구르기 뒷구르기하고 봐도 든든한 팀장이었던 준형!
부족한 코드를 꼼꼼하게 리뷰해줘서 고맙고 앞으로의 개발 인생을 응원한다.

👩‍💻 민지에게

회사 일이랑 병행하느라 엄청 힘들었을텐데 맡은 부분 책임감있게 마무리해준거 너무 고마워!
더 자주 만나고 이야기해봤으면 좋았을 것 같은데 그러지 못해서 아쉽다ㅠ
갓기업 신입사원 축하하고 같은 팀으로 일할 수 있어서 너무 좋았다ㅠㅠ
경험삼아 지원한 대기업 그냥 가볍게 뚫어버린,, 그저 GOAT
회사 다니면서 합숙도 하느라고 시간 내기 어려웠을텐데 그동안 작업하느라 너무 고생했어!!

👨‍💻 좌훈이에게

새벽에도 멈추지 않는 pr로 항상 나를 놀라게 했던 새벽좌 좌훈 오빠ㅋㅋㅋ 
프로젝트 하면서 필요한 부분을 잘 캐치해서 팀원들에게 전달해줘서 항상 고마웠어.
기회가 된다면 다같이 계속 네트워킹하자ㅠㅠ
뒤에서 묵묵히 팀에 필요한게 무엇인지 생각하고 손수 만들어 제공하는 츤데레 좌훈이형
코드도 잘쓰고 블로그도 잘쓰는 형이랑 같이 플젝하면서 많이 배웠고 다음 학기에 자주 보자!

team3-server's People

Contributors

Stargazers

Watchers

Forkers

jwahoonkim leeeryboy

team3-server's Issues

구글 로그인 관련 버그

구글 계정을 선택하고, 회원 정보를 기입하는 페이지로 넘어가기 직전에 토스트로 회원가입이 필요하다는 알림이 뜹니다. 뭔가 화면이 넘어가기 전에 회원가입이 돠어 있는지의 변수를 확인한 거 같은데, 앱 플로우 확인 한번 해주시면 좋을 것 같아요!

웹소켓 관련 약간의 취약점

로그인 없이 웹소켓 연결이 가능합니다
이에 따라 roomUUID를 알면 로그인 없이 해당 채팅에 메세지를 보내거나 받는게 가능해지네요
아마 STOMP도 핸들러를 상속받아 커스텀해서 access token을 검사한다거나 하는게 가능했을 것 같습니다!

아래는 postman으로 테스트 해본 사진입니다

[BUG] 회원가입 시 유저 이름 중복확인 비어있을 때 문제

유저 이름을 입력하지 않았을 때 중복 확인을 누르면 위의 사진과 같이 사용 가능한 닉네임이라고 뜹니다.

동네생활/검색 기능 관련 오류

pagenation이 2페이지 이후일때 검색하면 검색어에 해당하는 동네생활글이 있음에도 검색이 되지않습니다.

export const requestNeighborhood = async (
  accessToken: string,
  page: number,
  keyword: string,
) => {
  try {
    if (keyword === '') {
      return await axiosI.get(`/neighborhood/?page=${page}`, {
        headers: auth(accessToken),
      });
    } else {
      return await axios.get(
        `${BASE_URL}/neighborhood/?page=${page}&keyword=${keyword}`,
        {
          headers: auth(accessToken),
        },
      );
    }
  } catch (e) {
    return e;
  }
};

keyword가 있을 때 /neighborhood/?page=${page}&keyword=${keyword} 로 api콜을 보내는데
keyword 변경 시 page를 default로 set 하는 useEffect가 있어야할 것 같습니다.

[BUG] 가격 입력 관련 버그

중고거래 글을 쓸 때, 가격이 일정 수 이상이 되면 아래와 같은 형식으로 변합니다.

여기에 0을 더 입력하면 아래처럼 되는데, 의도하신 부분은 아닌 것 같아 알려드립니다.

거래완료 물품 후기보기 시, 내가 구매한 물품이 아닌데 후기 작성자 닉네임이 내 닉네임으로 뜨는 현상

username 관련 설계가 잘못된 것 같습니다.

                {buyerReview.id && (
                  <ReviewInfo
                    userId={buyer.id}
                    img={buyer.imgUrl ? buyer.imgUrl : defaultImg}
                    username={buyer.username}
                    type={'BUYER'}
                    location={buyer.location}
                    createdAt={buyerReview.createdAt}
                    content={buyerReview.content}
                  />
                )}

buyer라는 변수가 잠재적 buyer인 사용자 본인을 지칭하는 것 같은데 그걸 그대로 넘겨줘서
구매완료 상품일때 실 buyer가 아닌 여전히 사용자 본인이 넘어가네요

그런데 나의 와플->My chats 에 해당 채팅목록이 뜨는 오류가 발생합니다.