g-noon / advanced-fan-mail-box Goto Github PK
View Code? Open in Web Editor NEW인증/인가 기능이 추가된 Fan-Mail-Box 제작 프로젝트(Current: Fail)
License: MIT License
인증/인가 기능이 추가된 Fan-Mail-Box 제작 프로젝트(Current: Fail)
License: MIT License
안녕하세요, 진훈님 벌써 심화주차 리뷰를 하게 되었네요. 바로 시작해 볼게요. :)
advanced-fan-mail-box/src/pages/Login.jsx
Lines 28 to 65 in 1d929e8
저는 진훈님이 작성하신 것처럼 사용하는 쪽에서 성공/에러에 대한 처리를 해주는 것을 좋아해요. 서버와의 통신에 대한 로직은 숨기고, 성공이나 실패 시 어떤 메시지를 띄울지는 최종 사용자에게 맡기는 거죠. 다만, 코드 가독성을 위해 회원가입과 로그인에 대한 로직을 함수로 따로 빼는 건 어떨까요?
advanced-fan-mail-box/src/shared/redux/modules/mailListSlice.js
Lines 11 to 16 in 1d929e8
깃헙 저장소에 최종으로 제출 시, console.log()가 있는지 꼭 확인하고 제출하시는 것을 추천드릴게요.(꼭 필요한 게 아닌 개발하면서 확인을 위해 사용한 console.log()) 나중에 기술 과제를 볼 때 감점 사유가 될 수 있어요.
advanced-fan-mail-box/src/pages/Detail.jsx
Lines 8 to 10 in 1d929e8
mailList
는 객체이기 때문에 배열 메서드인 find
사용 시 에러가 납니다! mailList
의 letters
를 의도하신 걸까요?
advanced-fan-mail-box/src/api/axiosAPI.js
Lines 3 to 17 in 1d929e8
axiosAPI를 named export 하셨는데, 불러오는 쪽에서는 그렇지 못한 것 같아요. 저 같은 경우에는 바로 에러가 떴는데 혹시 이상 없으셨을까요?
이렇게 피드백을 드려봤습니다. 저번 리뷰 때 디렉토리 구조에 대해서 얘기를 나눈 적이 있었죠. 고민의 흔적이 보이네요. 🧐 이번 아웃소싱 프로젝트도 화이팅입니다! 고생하셨습니다. 👏
A declarative, efficient, and flexible JavaScript library for building user interfaces.
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
An Open Source Machine Learning Framework for Everyone
The Web framework for perfectionists with deadlines.
A PHP framework for web artisans
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
Some thing interesting about web. New door for the world.
A server is a program made to process requests and deliver data to clients.
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
Some thing interesting about visualization, use data art
Some thing interesting about game, make everyone happy.
We are working to build community through open source technology. NB: members must have two-factor auth.
Open source projects and samples from Microsoft.
Google ❤️ Open Source for everyone.
Alibaba Open Source for everyone
Data-Driven Documents codes.
China tencent open source team.