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] Shoot 생성 API 구현, 상태 변경 수정 #64

Merged
merged 7 commits into from
Nov 19, 2024
Merged

Conversation

cowboysj
Copy link
Member

@cowboysj cowboysj commented Nov 18, 2024

#️⃣ 관련 이슈

💡 작업내용

1. ShootService에서 ShootStatus 관련 메서드 ShootStatusService로 분리하였습니다.

  • Our Shoot 조회는 쿼리 파라미터로 4개의 탭을 다 구현할 예정이어서 ShootStatusController를 만들게 되면 2개의 API만 나오게 될 것 같아 컨트롤러는 ShootController 한 개로 유지하였습니다.

2. 슛 상태를 업데이트 하는 로직을 수정했습니다.

  • figmaUtil을 불러오지 않았어서 수정하였습니다.
  • 해당 슛 정보(GetShootResponse)를 바로 반환하게 수정하였습니다.

3. 새로운 Shoot 생성 API 구현

  • 현재 프론트엔드 작업량이 많은 것 같아 백엔드에서 유저 언급(@sujin) 처리를 하는 게 좋을 것 같습니다.
  • shoot content를 넘겨줄 때도 그대로 string 하나로 주고, 프론트에서 @처리를 따로 해 색상을 변경하면 될 것 같습니다.
  • 슛 내용에 태그가 아닌 @ 그 자체가 들어갈 수 있는 경우는 일단 배제하는 게 좋을 것 같아요..!
  • 새로운 Shoot을 생성하면 그대로 content에 담아서 요청됩니다.
  • ShootUtil의 extractTags를 통해 @가 붙은 단어를 분리하는 로직을 작성하였습니다.
  • createShootTags를 통해 해당 슛의 태그된 figma user가 저장됩니다.
    ex) @SUJIN님~! 완료하셨나요? @지선
    이렇게 content가 오면, @ 붙은 걸 분리해서 SUJIN, 지선을 추출한 뒤 해당 피그마 이름이 있는지 확인하고 저장하게 하였습니다.

📸 스크린샷(선택)

📝 기타

(참고사항, 리뷰어에게 전하고 싶은 말 등을 넣어주세요)

@cowboysj cowboysj linked an issue Nov 18, 2024 that may be closed by this pull request
2 tasks
@cowboysj cowboysj self-assigned this Nov 18, 2024
@cowboysj cowboysj added the ✨ feature 기능 구현 label Nov 18, 2024
@cowboysj cowboysj requested a review from iiqcov November 18, 2024 17:19
Copy link
Contributor

@iiqcov iiqcov left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~! 지금 pr이 너무 커지는 것 같은데 이후에는 기능 단위로 pr을 올려도 좋을 것 같아요~

@cowboysj cowboysj merged commit f8c3b42 into develop Nov 19, 2024
2 checks passed
@iiqcov iiqcov deleted the feat/#60 branch November 23, 2024 12:52
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.

[Feat] Shoot 생성, Shoot Status 업데이트 수정
2 participants