issues
search
peoplefund-tech
/
react-naver-login
A Naver Login Component for React
MIT License
19
stars
9
forks
source link
코드 개선
#3
Closed
tmmoond8
closed
4 years ago
tmmoond8
commented
4 years ago
peerDependencies를 좀 더 낮출 수 있지 않을까?
react, react-dom 은 devDepencies 로 가야 하지 않을까?
popUp창에서 window.close를 호출 할 때 setInterval을 종료 해야 하지 않을까?
render는 필수 prop 인데 왜 존재를 체킹 할까?
tmmoond8
commented
4 years ago
onSuccess, onFailure 에 타입을 좀 더 명시 해주면 더 좋을 것 같은데!?