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.

Sign up for GitHub

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

Jump to bottom

공유 보관함 잠금 기능 #176

Open
Ji-Ha opened this issue Aug 14, 2022 · 2 comments
Open

공유 보관함 잠금 기능 #176

Ji-Ha opened this issue Aug 14, 2022 · 2 comments
Assignees
@Ji-Ha
Labels
Shared

Comments

@Ji-Ha
Copy link
Contributor

Ji-Ha commented Aug 14, 2022

보관함 잠금

  • 초대링크로 들어오는 사용자를 잠그는 기능
  • 읽기링크로 들어오는 사용자들을 잠그는 기능 추가
@Ji-Ha Ji-Ha self-assigned this Aug 14, 2022
@Ji-Ha Ji-Ha added the Shared label Aug 14, 2022
@Ji-Ha Ji-Ha changed the title 공유 보관함 접근제한 기능 공유 보관함 잠금 기능 Aug 14, 2022
@Ji-Ha
Copy link
Contributor Author

Ji-Ha commented Aug 14, 2022 •
edited
Loading

folder에 있는 shared 컬럼을 통하여 잠금 기능을 구현하도록 하면 어떨까?

  • INVITE
  • OPEN
  • INVITE & OPEN
  • CLOSED

ENUM으로 사용하고, 추가될 컬럼이 없는지 회의해볼 것.

INVITE -> 초대 링크를 추가할 때.
OPEN -> 비회원도 접속이 가능한 링크를 뿌릴 때.
INVITE & OPEN -> 비회원도 접속이 가능하고, 초대 링크를 추가할 수 있을 때
CLOSED -> 보관함 잠금 상태

만약 그러면 CLOSED 상태에서 다시 변경하는 경우에는 이전 상태로 복원이 되어야하죠? -> 맞음

@Ji-Ha
Copy link
Contributor Author

Ji-Ha commented Aug 16, 2022

만약에 이메일 초대의 경우에는 그냥 사용자가 도토리함 첫 화면 들어왔을 때 알림이 가는 게 나은지 아니면 이메일을 직접 보내는 게 나은지?

Ji-Ha added a commit that referenced this issue Aug 16, 2022
@Ji-Ha
[#176] init(SharedType) : 보관함 상태 컬럼 추가
e508844
Ji-Ha added a commit that referenced this issue Aug 16, 2022
@Ji-Ha
Merge branch 'feature/#168-공유_관련_수정' of https://github.com/YAPP-19th/…
b737e04 
…Web-Team-2-Backend into feature/#176-공유_보관함_잠금_기능
Ji-Ha added a commit that referenced this issue Aug 16, 2022
@Ji-Ha
[#176] feature(보관함 잠금/해제)
ff317ac 
- SharedType에서 잠금해제 하는 기능 추가
- 폴더를 OPEN/INVITE/INVITE_AND_OPEN 상태로 변경하도록 하는 기능 추가
- ENUM 타입을 데이터베이스에 저장할 때 String 으로 저장할 수 있도록 수정
- 초대를 수락할 때, 보관함의 공유 상태확인 로직 추가
- 비회원이 보관함을 조회할 때, 보관함의 공유 상태확인 로직 추가
Ji-Ha added a commit that referenced this issue Aug 21, 2022
@Ji-Ha
Merge branch 'develop' into feature/#176-공유_보관함_잠금_기능
080edf8
Ji-Ha added a commit that referenced this issue Aug 23, 2022
@Ji-Ha
Merge pull request #177 from YAPP-19th/feature/#176-공유_보관함_잠금_기능
66f7d22 
[#176] 공유 보관함 잠금 기능
Ji-Ha added a commit that referenced this issue Sep 20, 2022
@Ji-Ha
[#176] fix(보관함 잠금/해제)
5e8948f 
- SharedType 구성 변경
- 보관함 잠금/해제 로직 변경
- ENUM 타입에 맞게 공유 링크 생성하도록 변경
- 링크를 통한 북마크 조회 변경
- 초대링크를 통한 공유보관함 초대 로직 변경
Ji-Ha added a commit that referenced this issue Sep 20, 2022
@Ji-Ha
Merge branch 'develop' into feature/#176-공유_보관함_잠금_기능
29dae36
Ji-Ha added a commit that referenced this issue Sep 26, 2022
@Ji-Ha
Merge pull request #182 from YAPP-19th/feature/#176-공유_보관함_잠금_기능
c0aa331 
[#176] 공유보관함 잠금/해제 기능 변경 & 링크 생성 API 변경
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Ji-Ha Ji-Ha

Labels
Shared
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Ji-Ha