Skip to content

Conversation

@jeeheaG
Copy link
Contributor

@jeeheaG jeeheaG commented Mar 6, 2025

작업한 내용을 설명해주세요 ✔️

아래 세가지 api 를 구현했습니다.

  1. update crew
  2. update crew join rule
  3. close crew
    • 크루 폐쇄는 단순히 CrewStatus 를 SUSPENDED 로 변경하도록 구현되어있습니다. delete 처리와 다르다고 오인한 채로 구현했던 것이어서, 리뉴얼 후에 이 기능이 남아있게 된다면 제가 수정하도록 하겠습니다.

이외 특이사항은 없습니다.
작업한 시점에 비해 PR 이 너무 늦었네요..

감사합니다.

확인하기

  • : 코드에 에러가 없는지 확인했나요?
  • : PR에 설명을 기재했나요?
  • : PR 태그를 붙였나요?
  • : 불필요한 로그나 System.out을 제거했나요?

@jeeheaG jeeheaG self-assigned this Mar 6, 2025
@jeeheaG jeeheaG added the ⚒️ enhancement ⚒️ New feature or request label Mar 6, 2025
@ekgns33 ekgns33 self-requested a review March 6, 2025 16:09
Copy link
Collaborator

@ekgns33 ekgns33 left a comment

Choose a reason for hiding this comment

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

고생하셨어용~ 질문하나만 남겼습니당

@jeeheaG jeeheaG merged commit 8e73120 into main Mar 7, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⚒️ enhancement ⚒️ New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants