프론트엔드 5기 8조 과제 (김필진, 유희태, 박철민, 차동민) #5
Conversation
This file contains hidden or 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
cdm1263
changed the title
happyhermann
left a comment
happyhermann
left a comment
There was a problem hiding this comment.
코드 리뷰가 늦어서 죄송합니다
완성된 것을 현업에 있는 분들한테 보여줬더니
너무 잘했다고 칭찬을 하더라고요
코드는 물론이고 무엇보다 디자인과 UI를 보고 당장 상용에 올려도 괜찮을 정도라고..
코드가 매우 클린하고 잘 짜여있어서 보면서 정말 현업에 가시면
급격히 성장하실거 같다는 생각이 들었습니다
처음에 했던 프로젝트도 좋았지만 프로젝트가 진행될 수록 퀄리티가 올라가는 거 같네요
결국엔 리액트 사가까지 적용하고 타입 스크립트도
자칫 @ts-ignore과 any를 찾게 되면서 Type이 덕지덕지 붙은 코드로 채워지길 마련인데
스키마 interface도 최대한 쓰려고 노력한것 같네요
타입스크립트를 공부하면서 느낀건 타입선언을 잘하는 것도 좋지만
가급적 자동추론을 사용하도록 노력하면 좀더 좋은 타입스크립트 생태계를 만들 수 있는 것 같습니다
또한 웹스토리지 객체를 적절하게 써주신 바도 인상 깊었습니다
Comment on lines
+21
to
+27
| const isAdmin = async () => { | ||
| const res = await check(); | ||
| if (!(res.email === ADMIN_EMAIL && res.displayName === ADMIN_NAME)) { | ||
| alert("관리자가 아닙니다."); | ||
| navigate("/"); | ||
| } | ||
| }; |
There was a problem hiding this comment.
어드민 페이지 리다이렉트를 잘 해주셨는데요 경로 이동 히스토리가 남는 경우엔 뒤로가기를 해도 원하는 페이지로 이동하지 않는 경우가 있는데 이런 경우엔 redirect 메서드를 사용해보시는 것도 좋을거 같습니다!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
🪐 우주부동산
관리자 계정
👩🚀 개발팀
유희태
김필진
박철민
차동민
임무 분담
사용자 페이지
인증-인가 페이지
관리자 페이지
상품 페이지
사용기술 및 개발환경
Development
Config
Deployment
Environment
Cowork Tools
설치
개발 서버
다음 명령어를 입력하세요.
전체 화면 구성
메인 페이지
메인 페이지
메인 페이지 - 태그 검색
메인 페이지 - 검색창 검색
상세 페이지
장바구니
주문 페이지
마이 페이지 - 주문 내역
마이 페이지 - 계좌 관리
마이 페이지 - 찜한 상품
마이 페이지 - 개인정보 수정
로그인
회원 가입
관리자 페이지 - 전체 거래 내역
관리자 페이지 - 사용자 조회
관리자 페이지 - 모든 제품 조회
