OrangeTen / TodayILearned

http://tilup.sullivan.kr
MIT License
40 stars 4 forks source link

Feature/wrap up express rebase #80

Closed kujyp closed 6 years ago

kujyp commented 6 years ago
  1. ESLint plugin dependencies 정리, npm run lint script추가

  2. Access-Control-Allow-Headers에 Authorization 추가 클라이언트단에서 Request header field Access-Control-Allow-Origin is not allowed by Access-Control-Allow-Headers in preflight response문제를 야기함.(포스트맨에서는 볼수없는 이슈)

  3. React static routes추가(til/, repo/, profile/)

  4. PR #78 에서 중복커밋이 그대로 들어가는것 확인하고, 다시 dev에서 끌어온 후에 변경된 3개의 커밋만 추가

Loloara commented 6 years ago
  1. npm run lint 스크립트는 그러면 프로젝트 전체 eslint 검사 및 수정되는건가요? json으로 박혀있는 쿼리문 같은 경우 괄호들이 펼쳐져서 보기 끔찍해지는데ㅠ
  2. 넵 감사합니다
  3. 서버는 api/ 라우팅만 사용해서 나머지는 마음껏 쓰시면 됩니당
kujyp commented 6 years ago

@Loloara

  1. 아뇨 수정은 안합니다. 근데 서버폴더에 .eslintrc.js 있길래, 검사만 추가한거뿐이에요 ㅋㅋㅋㅋ 맘에안드시면 rule변경하시거나, 제거하시면!! 물론 제거보다는 rule변경이 더 좋을거같아요

https://github.com/OrangeTen/TodayILearned/blob/d90d5775ab6347976d246fe590f1a8a2d17dcd36/tilup-server/.eslintrc.js#L2