Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
bokeeeey pushed 1 commit to 62-be-refactor-authroute-수정 codeit-internship-group-b/codeit-resources
- BE-Feat: userRouter => update, delete 추가 ac6d3e0
cindycho0423 pushed 1 commit to 59-fe-feat-좌석예약페이지-ui-제작 codeit-internship-group-b/codeit-resources
- FE-Refactor: context로 프롭 드릴링 해결 ec56eb1
bokeeeey created a review on a pull request on codeit-internship-group-b/codeit-resources
고생햇어 LGTM~!
bokeeeey created a review comment on a pull request on codeit-internship-group-b/codeit-resources
Ayden에게 null반환은 좋지않은 사용이라고 피드백 들었던거같아 해당 부분은 내일 사무실에서 조언을 요청해볼까
bokeeeey created a review comment on a pull request on codeit-internship-group-b/codeit-resources
```ts import { Router } from "express"; import asyncHandler from "express-async-handler"; import { getUsers, getUser, createUser } from "../controllers/userController"; import { upload } from "...
bokeeeey created a review comment on a pull request on codeit-internship-group-b/codeit-resources
이것도 신경쓰이면 그냥 삭제해 나중에 내가 merge할때 제대로 구현해서 넣을게
bokeeeey created a review comment on a pull request on codeit-internship-group-b/codeit-resources
이 파일은 삭제해도 좋을듯 이제 안써!
bokeeeey created a review comment on a pull request on codeit-internship-group-b/codeit-resources
type확인하는 것도 유틸함수 안으로 넣어도 될거같은데 아니면 searchDate + "" 처리해서 사용가능하게 넘겨도 좋을거같고
bokeeeey created a review comment on a pull request on codeit-internship-group-b/codeit-resources
userModel이랑 동일파일이라 삭제해도 좋을듯?
bokeeeey created a review comment on a pull request on codeit-internship-group-b/codeit-resources
 ignore에 설정해놔서 파일명 맞추면 git에 안올라올거같아 테스트파일은 가급적 올리지말자
bokeeeey created a review comment on a pull request on codeit-internship-group-b/codeit-resources
unknown type이 사용처에서 제대로 캐스팅되고있지 않은거같은데...?
bokeeeey created a review comment on a pull request on codeit-internship-group-b/codeit-resources
전체적으로 interface로 정리해서 쓰는게 좋을거같아보이는데... 급하면 나중에 다듬어도 좋을듯해
bokeeeey created a review comment on a pull request on codeit-internship-group-b/codeit-resources
type이 뭔가 어색해보이는데...
AdamSeungheonShin created a review comment on a pull request on codeit-internship-group-b/codeit-resources
인터페이스 선언하면서 nullable 옵션을 고려 안해서 일단 에러 방지 용도로 해뒀는데 이미지 없는 경우에 어떻게 처리할지 얘기 해보자~!
cindycho0423 created a review comment on a pull request on codeit-internship-group-b/codeit-resources
hoxy 이미지가 없으면 null 로 안보내고 빈 문자열로 보내는 이유가 모야?? 보통 이미지 없을 때를 null 로 받아서 처리해줬던 것 같아서!!
cindycho0423 created a review on a pull request on codeit-internship-group-b/codeit-resources
고생했어,, 아는게 없어서 리뷰를 못남기네,,,
cindycho0423 created a review on a pull request on codeit-internship-group-b/codeit-resources
고생했어,, 아는게 없어서 리뷰를 못남기네,,,
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
@AdamSeungheonShin, 알겠습니다. 말씀하신 내용 확인하였으며, 앞으로 리뷰 시 참고하겠습니다. (◕‿◕✿) 궁금한 점이 해결되었네요! --- <details> <summary>✏️ Learnings added</summary> ``` Learnt from: AdamSeungheonShin PR: codeit-internship-g...
AdamSeungheonShin created a review comment on a pull request on codeit-internship-group-b/codeit-resources
프론트에서 날짜를 텍스트로 표시해줄때만 로컬타임으로 계산해서 처리하고 나머지 로직에 활용되는 날짜 데이터들은 모두 UTC로 사용하기로 했어
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
@AdamSeungheonShin, 네, 특정 모델에서 `type` 필드를 명시적으로 지정하고 싶으신 경우라면, 하위 스키마에서 `type` 필드를 정의하셔도 됩니다. 다만 Mongoose의 디스크리미네이터를 사용할 때는 `type` 필드를 디스크리미네이터 이름으로 자동 설정하므로, 추가적인 정의 없이도 원하는 값을 가질 수 있습니다. 예를 들어, ...
AdamSeungheonShin created a review comment on a pull request on codeit-internship-group-b/codeit-resources
특정 모델에 type 필드를 텍스트로 지정해주고 싶어서 그런건데 이렇게 하면 안되나?
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
@AdamSeungheonShin, 네, 말씀하신 대로 `findByIdAndUpdate`를 사용하면 요청 본문에 포함된 데이터 필드만 업데이트되고, 기존 필드들은 유지됩니다. 확인해주셔서 감사합니다. （。・＿・。） --- <details> <summary>✏️ Learnings added</summary> ``` Learnt from: A...
AdamSeungheonShin created a review comment on a pull request on codeit-internship-group-b/codeit-resources
mongoDB에서 body로 전달하는 데이터필드만 알아서 수정해주고 기존 필드들은 유지해주는걸로 알고 있는데