[과제] 모각코 7조 조항범 과제 제출 #11
Conversation
| @@ -15,7 +15,7 @@ function App() { | |||
|
|
|||
| const handleReloadClick = async (team) => { | |||
| try { | |||
| const result = await axios.get("http://127.0.0.1:8000/dust"); | |||
| const result = await axios.get("http://127.0.0.1:3000/dust/" + toggle); | |||
There was a problem hiding this comment.
보통 React는 3000번 포트를 기본으로 웹서버를 열고 있습니다! 그래서 react 서버의 포트를 바꾸거나, express 서버의 포트를 8000번으로 수정하면 좋을 것 같습니다!
|
|
||
| app.use(cors()) // CORS 를 사용하여 조건을 통하여 요청을 허용/비허용할 수 있다. 비어있으면 전부 허용 | ||
|
|
||
| app.get('/dust/:toggle', (req, res) => { // req : requset, res : response |
There was a problem hiding this comment.
params를 사용했는데, 여기에서는 필요하지 않은 기능인것 같습니다!
--> API 명세서에 포함되지 않은 내용입니다!
There was a problem hiding this comment.
아마, 코드를 local에서 모두 작업하고 파일 하나만 추가로 올린 것 같은데, 이후에 플젝을 진행하게 될때 node_modules를 제외한 나머지 파일들(package.json, package-lock.json)이 존재한다면, 모듈들의 버전 등을 맞출 수 있게 됩니다! 제가 .gitignore에 node_modules를 추가해놓았으니 git clone을 통해서 local에서 작업하시고, git add, git commit, git pull을 통해서 자신이 fork한 레포에 올리고, pull request를 걸어주면 많은 도움이 될 것입니다!!
나머지는 잘하신것 같아요~ 굳굳
| res.json([{'team':"모각코 7조", 'value':"1000"}, {'team':"모각코 2조", 'value':"38"}]) | ||
| }) | ||
|
|
||
| app.patch('/toggle/:toggle', (req, res) => { // req : requset, res : response |
There was a problem hiding this comment.
이 부분은 API 명세서에 표기를 하지 않았는데도, 잘 수행하신 것 같습니다!!
toggle을 backend server가 아닌, frontend server에서 다루게 했는데, 지금은 아직 알려드리지 않았지만 이후에는 backend server 쪽에서 이를 다루게 될 것입니다!
굳굳
There was a problem hiding this comment.
리뷰 감사합니다~ 레포는 말씀해주신대로 git clone으로 작업하도록 하겠습니다! 확인해주셔서 감사합니다 :)
팀원들과 협업하여 성공적으로 했읍니다,,,
코드가 이상할 수 있습니다 ㅜㅜ
#4