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

타인의 템플릿 페이지에서 노출된 업로드 버튼 오류 수정 #974

Merged
merged 4 commits into from
Dec 18, 2024

Conversation

healim01
Copy link
Collaborator

⚡️ 관련 이슈

#973

📍주요 변경 사항

주요 변경 사항에 대해 작성해주세요.

기존 타인의 템플릿에서 카테고리들 중 템플릿이 없으면 업로드 버튼이 보이고 있었습니다.
기존 코드에서 이미 isMine 이라는 조건으로 edit 모드 처리를 하고 있어서 해당 조건을 사용해서 처리했습니다.

추가적으로 모킹 수정하고 반영되지 못한 수정사항이나 sort-option 등이 몇개 있어서 같이 처리해서 올렸습니다~

🎸기타

고려해야 하는 내용을 작성해 주세요.

고민 중에 있는 부분은 MyTemplatePage 이라는 워딩 부분인데요,,~!

사실상 url 로 맴버 템플릿이고 본인 템플릿과 더불어 다른 사람의 템플릿을 모아보기도 가능한 기능이라 MemberTemplatePage 로 변경하는 것에 대해서 어떻게 생각하시는지 궁금합니다,,~~
폴더 명도 MyTemplatePage 이다보니 diff 가 많이 생길거 같아서요!

더불어 엠플리튜드 트래킹도 [Viewed] 내 템플릿 페이지 로 찍히고 있어서 제가 마위의 템플릿들을 구경해도 제 페이지를 보고 있는 걸로 찍힐거 같아서 수정이 필요할거 같습니다!

🍗 PR 첫 리뷰 마감 기한

12/20 12:00

@healim01 healim01 added bug 개발자가 의도하지 않은 상황 FE 프론트엔드 labels Dec 17, 2024
@healim01 healim01 self-assigned this Dec 17, 2024
Copy link
Contributor

@Jaymyong66 Jaymyong66 left a 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');
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 요고 스크립트로 얼른 만들어야겠네여

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

사수가 또 일 준다....

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

해당 작업까지 진행해서 새 PR 올려놨습니다...~
새벽 3시인데 마위 때문에 못잔 헤일리의 PR 보러가기

@Jaymyong66 Jaymyong66 added the zap 리뷰 우선순위가 높은 사항 label Dec 17, 2024
Copy link
Contributor

@Hain-tain Hain-tain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

새벽에 고생하셨습니다👍 파일명 변경은 너무 좋아요! 이미 변경사항은 다 확인한 후라 이번 PR에서 해도 좋고, 모노레포 작업 후 한번에 해도 좋습니다.

@Jaymyong66 Jaymyong66 merged commit 093bf5a into dev/fe Dec 18, 2024
7 checks passed
@Jaymyong66 Jaymyong66 deleted the fix/973-unshow-upload-button branch December 18, 2024 09:29
@healim01 healim01 added this to the 7차 스프린트 💭 milestone Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 개발자가 의도하지 않은 상황 FE 프론트엔드 zap 리뷰 우선순위가 높은 사항
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants