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

[FIX] 현재 완전히 구현되지 않은 기능 UI 제거 #576

Merged
merged 1 commit into from
Nov 25, 2024

Conversation

dmdgpdi
Copy link
Collaborator

@dmdgpdi dmdgpdi commented Nov 25, 2024

What is this PR? 🔍

Changes 📝

  • 현재 완전히 구현되지 않아 버그가 발생하거나 예상하지 못한 동작을 수행하는 기능(웹 내에서 픽 생성 및 검색)의 UI를 제거합니다.

Precaution

@dmdgpdi dmdgpdi added frontend 프론트 관련 fix 🔥 bug 수정을 포함한 변경 사항 labels Nov 25, 2024
@dmdgpdi dmdgpdi self-assigned this Nov 25, 2024
@dmdgpdi dmdgpdi merged commit 883bae0 into frontend Nov 25, 2024
1 check passed
@dmdgpdi dmdgpdi deleted the front/fix/#575 branch November 25, 2024 01:27
dmdgpdi added a commit that referenced this pull request Nov 25, 2024
* [FIX] 원래 코드 복구 (#568)

* fix: 기존 작업 코드 복구

* fix: build 위해 안쓰는 파일 주석

* [DESIGN] 픽 생성시 zIndex 추가 (#570)

* [FIX] 현재 완전히 구현되지 않은 기능 UI 제거 (#576)

---------

Co-authored-by: kimminkyeu <[email protected]>
dmdgpdi added a commit that referenced this pull request Nov 25, 2024
* fix: 깃 충돌 방지를 위한 테스트 커믹

* fix: 충돌 방지를 위한 테스트 커밋

* fix: 깃 머지 테스트를 위한 commit

* fix: 깃 머지 테스트를 위한 commit

* fix: 깃 머지 테스트를 위한 commit

* fix: 깃 머지 테스트를 위한 commit

* [FIX] 원래 코드 복구 (#568)

* fix: 기존 작업 코드 복구

* fix: build 위해 안쓰는 파일 주석

* [DESIGN] 픽 생성시 zIndex 추가 (#570)

* [FIX] 현재 완전히 구현되지 않은 기능 UI 제거 (#576)

* [FIX] PickRecord에서 태그가 보이지 않는 문제 수정 (#580)

* refactor: PickTagPicker 이름 변경

* feat: PickRecord에서 수정시 더블 클릭되게 변경

* fix: 페이지 변경시마다 tagList 불러오기

---------

Co-authored-by: kimminkyeu <[email protected]>
dmdgpdi added a commit that referenced this pull request Nov 29, 2024
* [FIX] 사용하지 않는 기능 UI 제거 (#577)


* [FIX] 원래 코드 복구 (#568)

* fix: 기존 작업 코드 복구

* fix: build 위해 안쓰는 파일 주석

* [DESIGN] 픽 생성시 zIndex 추가 (#570)

* [FIX] 현재 완전히 구현되지 않은 기능 UI 제거 (#576)

---------

Co-authored-by: kimminkyeu <[email protected]>

* [FIX] 현재 PickRecord에서 조회 안되는 문제 수정 (#581)

* fix: 깃 충돌 방지를 위한 테스트 커믹

* fix: 충돌 방지를 위한 테스트 커밋

* fix: 깃 머지 테스트를 위한 commit

* fix: 깃 머지 테스트를 위한 commit

* fix: 깃 머지 테스트를 위한 commit

* fix: 깃 머지 테스트를 위한 commit

* [FIX] 원래 코드 복구 (#568)

* fix: 기존 작업 코드 복구

* fix: build 위해 안쓰는 파일 주석

* [DESIGN] 픽 생성시 zIndex 추가 (#570)

* [FIX] 현재 완전히 구현되지 않은 기능 UI 제거 (#576)

* [FIX] PickRecord에서 태그가 보이지 않는 문제 수정 (#580)

* refactor: PickTagPicker 이름 변경

* feat: PickRecord에서 수정시 더블 클릭되게 변경

* fix: 페이지 변경시마다 tagList 불러오기

---------

Co-authored-by: kimminkyeu <[email protected]>

* feat: add context menu item 공유하기

- icon, text 추가. 기능 x

* feat: share folder api

- 스키마 정의, res는 아직 수정중이라셔서 any로 받음
- api 호출, 에러핸들링 관련 명세가 없어서 로그만 남김

* refactor: shareFolder API 함수 콜 로직 변경

- 기존: store에 정의된 shareFolderById()를 호출하여 shareFolder API 콜
- 변경: 뷰에서 shareFolder API를 바로 호출
- 이유: 뷰의 props로 id만 주어진 줄 알았는데 requestDTO에 필요한 name도 함께 주어져 store에 비지니스 로직을 따로 둘 필요가 사라짐

* feat: Share Folder Dialog

- 공유하기 다이얼로그 팝업
- Copy 버튼 클릭시 uuid를 포함한 share link 복사
- "내설정" 링크 클릭시 공유된 폴더 링크로 이동(라우팅 경로 아직 안만듦)

* feat: routing share page

- /share/[uuid]로 라우팅 되도록 라우팅 디렉토리 생성

* fix: Share Folder Link 경로 수정

- 기존: domain/folder/uuid
- 변경: domain/share/uuid

* chore: import 순서

---------

Co-authored-by: dmdgpdi <[email protected]>
Co-authored-by: kimminkyeu <[email protected]>
obvoso added a commit that referenced this pull request Dec 5, 2024
* [FIX] 사용하지 않는 기능 UI 제거 (#577)


* [FIX] 원래 코드 복구 (#568)

* fix: 기존 작업 코드 복구

* fix: build 위해 안쓰는 파일 주석

* [DESIGN] 픽 생성시 zIndex 추가 (#570)

* [FIX] 현재 완전히 구현되지 않은 기능 UI 제거 (#576)

---------

Co-authored-by: kimminkyeu <[email protected]>

* [FIX] 현재 PickRecord에서 조회 안되는 문제 수정 (#581)

* fix: 깃 충돌 방지를 위한 테스트 커믹

* fix: 충돌 방지를 위한 테스트 커밋

* fix: 깃 머지 테스트를 위한 commit

* fix: 깃 머지 테스트를 위한 commit

* fix: 깃 머지 테스트를 위한 commit

* fix: 깃 머지 테스트를 위한 commit

* [FIX] 원래 코드 복구 (#568)

* fix: 기존 작업 코드 복구

* fix: build 위해 안쓰는 파일 주석

* [DESIGN] 픽 생성시 zIndex 추가 (#570)

* [FIX] 현재 완전히 구현되지 않은 기능 UI 제거 (#576)

* [FIX] PickRecord에서 태그가 보이지 않는 문제 수정 (#580)

* refactor: PickTagPicker 이름 변경

* feat: PickRecord에서 수정시 더블 클릭되게 변경

* fix: 페이지 변경시마다 tagList 불러오기

---------

Co-authored-by: kimminkyeu <[email protected]>

* rename: favicon.ico 파일 위치 변경

- page 내에서 segment를 props로 받아올 때 파비콘이 /public에 위치해 있지 않아서 위치한 경로를 찾으려고 들어오는 거 같습니다(추측..)
- /public으로 옮기고 metadata에서 사용할 때 절대경로로 수정

* feat: share folder page

- (domain)/share/uuid 페이지 렌더링

* rename: (signed)/folders/layout.tsx를 상위 디렉토리로 이동

- 와이어프레임에 마이페이지에서도 동일한 사이드바(폴더트리)를 사용해서 layout파일을 한단계 위로 올렸습니다

* feat: 폴더 공유

- 공개 링크로 접속시 나오는 페이지 생성
- 내 설정에서 폴더 공유 목록 확인 가능
- 내 설정에서 공유 취소 가능
- context menu에서 폴더 공개/비공개 가능
- 공유 폴더 비공개 요청 성공시 노티토스트 띄움
- 휴지통으로 이동시 나오는 다이얼로그에 공개 폴더 워닝 추가

* feat: 비공개 폴더 페이지에 접근시 EmptyPickRecordImage 렌더

* design: 공유중인 폴더를 아이콘으로 표시

- 피드백을 받고 "공유중"이라는 우측 텍스트 대신 좌측 아이콘으로 표시

* style: share folder관련 타입의 이름 변경

- ShareFolderReadFolderFullResponseType -> GetShareFolderListResponseType
- ShareFolderReadFolderPartialResponseType -> GetMyShareFolderResponseType

* fix: 폴더트리의 공유폴더 이름 변경시 마이페이지에 반영되지 않던 에러 해결

- 원인: treeDataMap을 디펜던시로 하는 useEffect에서 동일한 api엔드포인트로 요청을 보내 브라우저, 라이브러리(불명함..)에서 요청을 취소함
- 변경점: 1. 서버에서 fetch하는 대신 중앙store에서 공유폴더를 필터링하여 반환
2. useEffect 내부에서 로컬상태를 사용해 반환하던 훅에서 중앙 store의 상태가 변경될 때마다 공유 상태를 필터링하는 함수를 만들고 배열로 리턴하도록 변경

* rename: setShareFolder -> updateFolderAccessTokenByFolderId

---------

Co-authored-by: dmdgpdi <[email protected]>
Co-authored-by: kimminkyeu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix 🔥 bug 수정을 포함한 변경 사항 frontend 프론트 관련
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant