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

[#79] 게시글 태그 수정 리팩터링 #80

Merged
merged 4 commits into from
Jan 26, 2023
Merged

Conversation

hyena0608
Copy link
Collaborator

🌱 작업 사항

  • 게시글 수정시 태그 업데이트 메서드 리팩터링
      - 새로운 RootTag 존재시 저장한다.
      - 없어진 PostTag를 삭제한다.
        - UserTag 카운트를 줄인다.
          - UserTag 카운트가 0일시에 삭제한다.
      - 새로운 PostTag를 저장한다.
        - UserTag를 새로 저장하거나 카운트를 증가한다.

🦄 관련 이슈

close #79

- Post 엔티티 수정

- User 엔티티 수정

- PostTag 엔티티 수정

- RootTag 엔티티 수정

- UserTag 엔티티 수정
- 게시글 수정시 태그 업데이트 메서드 리팩터링
  - 새로운 RootTag 존재시 저장한다.
  - 없어진 PostTag를 삭제한다.
    - UserTag 카운트를 줄인다.
      - UserTag 카운트가 0일시에 삭제한다.
  - 새로운 PostTag를 저장한다.
    - UserTag를 새로 저장하거나 카운트를 증가한다.
@hyena0608 hyena0608 added the ♻️ refactor 코드 리팩터링 label Jan 26, 2023
@hyena0608 hyena0608 self-assigned this Jan 26, 2023
Copy link
Member

@JoosungKwon JoosungKwon left a comment

Choose a reason for hiding this comment

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

태그 기능은 끝나신건가요?

@hyena0608 hyena0608 merged commit 4e83181 into develop Jan 26, 2023
@hyena0608 hyena0608 deleted the refactor/79-post branch January 26, 2023 04:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ refactor 코드 리팩터링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

게시글 수정 시 태그 메서드 리팩터링
3 participants