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: 나무 삭제 #61

Merged
merged 5 commits into from
Jul 9, 2022

Conversation

kang-jisu
Copy link
Contributor

이슈

작업 내용

  • 나무 삭제, 삭제시 해당 나무에 포함된 메시지도 삭제처리 하도록 구현

기타 사항

  • 메시지-폴더 관계에서 폴더가 삭제되버리면 안될 것 같아서, 메시지 삭제할 때 모두 해당 유저의 기본 폴더로 옮기고 delByReceiver true로 바꾸는걸로 설정
  • 폴더 삭제
  • 기본 폴더는 삭제불가능

체크리스트

  • 테스트 통과

@kang-jisu kang-jisu added 💡feature API feature 🌲칭찬 칭찬 📬메시지함 메시지함 labels Jul 9, 2022
@kang-jisu kang-jisu requested a review from suubinkim July 9, 2022 14:46
@kang-jisu kang-jisu self-assigned this Jul 9, 2022
@kang-jisu kang-jisu added this to To do in BeTree via automation Jul 9, 2022
@@ -93,13 +93,14 @@ public void updateFavorite() {
/**
* 메세지 삭제 여부 상태 변경 메서드
*/
public void updateDeleteStatus(Long userId) {
public void updateDeleteStatus(Long userId, Folder defaultFolder) {
Copy link
Member

Choose a reason for hiding this comment

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

👍👍

@@ -113,15 +116,43 @@ public Long createTree(Long userId, TreeRequestDto treeRequestDto) {
*/
@Transactional
public void updateTree(Long userId, Long treeId, TreeRequestDto treeRequestDto) {
Folder folder = validateAndGetFolder(userId, treeId);
Copy link
Member

Choose a reason for hiding this comment

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

😍

@@ -20,4 +25,153 @@ public Claims parseToken(String token) {
};
};
}

public static Claims getClaims(Long userId) {
Copy link
Member

Choose a reason for hiding this comment

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

👍😁

@suubinkim
Copy link
Member

머지 고고 입니다

@kang-jisu kang-jisu merged commit 10e1fbc into YAPP-Github:develop Jul 9, 2022
BeTree automation moved this from To do to Done Jul 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌲칭찬 칭찬 💡feature API feature 📬메시지함 메시지함
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants