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

태그를 병합할 때, 서브 태그를 디폴트 태그로 다시 옮길 수 있게 함 #490

Merged
merged 5 commits into from
Jun 5, 2023

Conversation

nyj001012
Copy link
Contributor

개요

태그를 병합할 때, 서브 태그를 디폴트 태그로 다시 옮길 수 있게 함

작업 사항

슈퍼 태그 id로 null이 들어오면, 서브 태그들을 디폴트 태그로 옮김

변경점

  • /api/tags/merge API
  • 태그 컨트롤러에서 bookInfoId를 받아서 처리
  • 태그 서비스에서 id 값 유효성 검사 로직 변경

스크린샷 (optional)

superTagId가 null일 경우, 디폴트 태그로의 병합을 의미하도록 수정
슈퍼 태그와 서브 태그의 유효성 검사를 한 번에 하던 작업을 분리함
서브 태그를 디폴트 태그로도 옮길 수 있게 함
불필요한 함수 제거
@nyj001012 nyj001012 added the enhancement New feature or request label Jun 5, 2023
@nyj001012 nyj001012 requested a review from not-using June 5, 2023 07:32
@nyj001012 nyj001012 self-assigned this Jun 5, 2023
@nyj001012 nyj001012 linked an issue Jun 5, 2023 that may be closed by this pull request
3 tasks
@nyj001012 nyj001012 merged commit b326279 into develop Jun 5, 2023
@nyj001012 nyj001012 deleted the 453-태그-업데이트-기능-추가 branch June 5, 2023 07:41
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.

태그 업데이트 기능 추가
2 participants