[DESIGN] 픽 생성시 zIndex추가 #570
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
dmdgpdi
added a commit
that referenced
this pull request
Nov 22, 2024
* fix: 깃 충돌 방지를 위한 테스트 커믹 * fix: 충돌 방지를 위한 테스트 커밋 * fix: 깃 머지 테스트를 위한 commit * fix: 깃 머지 테스트를 위한 commit * fix: 깃 머지 테스트를 위한 commit * fix: 깃 머지 테스트를 위한 commit * [FIX] 원래 코드 복구 (#568) * fix: 기존 작업 코드 복구 * fix: build 위해 안쓰는 파일 주석 * [DESIGN] 픽 생성시 zIndex 추가 (#570) --------- Co-authored-by: kimminkyeu <[email protected]>
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
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What is this PR? 🔍
Changes 📝
zIndex추가해서 이쁘게 함
Precaution