-
Notifications
You must be signed in to change notification settings - Fork 8
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
타인의 템플릿 페이지에서 노출된 업로드 버튼 오류 수정 #974
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
변경사항이 명확해서 금방 리뷰했네요~!
저는 헤인이 한번 리뷰를 한 뒤에 MyTemplatePage
컴포넌트 명을 바꾸는건 해당 PR에서 진행해도 괜찮을거 같아요~
헤일리 짱~
@@ -35,7 +35,7 @@ const enableMocking = async () => { | |||
return; | |||
} | |||
|
|||
// const { worker } = await import('./mocks/browser'); | |||
// const { worker } = await import('@/mocks/settings/browser'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 요고 스크립트로 얼른 만들어야겠네여
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
사수가 또 일 준다....
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
해당 작업까지 진행해서 새 PR 올려놨습니다...~
새벽 3시인데 마위 때문에 못잔 헤일리의 PR 보러가기
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
새벽에 고생하셨습니다👍 파일명 변경은 너무 좋아요! 이미 변경사항은 다 확인한 후라 이번 PR에서 해도 좋고, 모노레포 작업 후 한번에 해도 좋습니다.
⚡️ 관련 이슈
#973
📍주요 변경 사항
기존 타인의 템플릿에서 카테고리들 중 템플릿이 없으면 업로드 버튼이 보이고 있었습니다.
기존 코드에서 이미
isMine
이라는 조건으로 edit 모드 처리를 하고 있어서 해당 조건을 사용해서 처리했습니다.추가적으로 모킹 수정하고 반영되지 못한 수정사항이나 sort-option 등이 몇개 있어서 같이 처리해서 올렸습니다~
🎸기타
고민 중에 있는 부분은
MyTemplatePage
이라는 워딩 부분인데요,,~!사실상 url 로 맴버 템플릿이고 본인 템플릿과 더불어 다른 사람의 템플릿을 모아보기도 가능한 기능이라
MemberTemplatePage
로 변경하는 것에 대해서 어떻게 생각하시는지 궁금합니다,,~~폴더 명도
MyTemplatePage
이다보니 diff 가 많이 생길거 같아서요!더불어 엠플리튜드 트래킹도
[Viewed] 내 템플릿 페이지
로 찍히고 있어서 제가 마위의 템플릿들을 구경해도 제 페이지를 보고 있는 걸로 찍힐거 같아서 수정이 필요할거 같습니다!🍗 PR 첫 리뷰 마감 기한
12/20 12:00