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

북마크 공유 관련 로직 수정 #168

Open
Ji-Ha opened this issue Jul 13, 2022 · 0 comments
Open

북마크 공유 관련 로직 수정 #168

Ji-Ha opened this issue Jul 13, 2022 · 0 comments
Assignees
Labels

Comments

@Ji-Ha
Copy link
Contributor

Ji-Ha commented Jul 13, 2022

  • 공유 북마크 이동에 필요한 로직들 수정
  • 모든 도토리 조회시에 뜨는 오류 -> AccountFolder의 Authority 컬럼 변화로 인한 문제
  • 북마크 조회시에 pagenation 이 되지 않는 오류 수정
  • 보관함 초대가 제대로 이루어지는지 확인
  • 초대한 사용자가 나가는 기능 추가
@Ji-Ha Ji-Ha added the Shared label Jul 13, 2022
@Ji-Ha Ji-Ha self-assigned this Jul 13, 2022
Ji-Ha added a commit that referenced this issue Jul 24, 2022
- 보관함 초대 API 추가
- folderIdList로 북마크를 조회하는 메소드 추가
- folder를 shared인 상태로 변경하는 로직 추가
- folder가 보관함인지 확인하는 로직 추가
Ji-Ha added a commit that referenced this issue Jul 24, 2022
- 공유 북마크에 대한 추가, 삭제, 이동, remind on/off, 업데이트 부분에서 사용자의 권한을 확인하는 API 추가
- shared 북마크에 접근하는 uri를 sharedBookmark에서 shared로 수정함
- Folder Entity에서 동일한 보관함인지 확인하는 로직 추가
Ji-Ha added a commit that referenced this issue Jul 24, 2022
- 북마크가 추가될 때, 동일한 url을 가진 북마크가 휴지통에 있을 경우, 확실한 오류사항을 인지할 수 있도록 exception 발생시킴.
Ji-Ha added a commit that referenced this issue Aug 12, 2022
Ji-Ha added a commit that referenced this issue Aug 12, 2022
- 클래스 내에 순서 변경
Ji-Ha added a commit that referenced this issue Aug 12, 2022
- folderToShared -> changeFolderToShared로 동사형식으로 변경함
Ji-Ha added a commit that referenced this issue Aug 12, 2022
- 권한 검사할 때, folder를 통해 검사하도록 변경함.
- remindOn할 때, remind가 존재하면 추가되지 않도록 변경함.
Ji-Ha added a commit that referenced this issue Aug 16, 2022
Ji-Ha added a commit that referenced this issue Aug 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant