Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
_:warning: Potential issue_ **예약 업데이트 시 중복 예약 검증 필요** 예약 시간(`startAt`, `endAt`)을 업데이트할 때, 새로운 시간대가 기존의 다른 예약과 겹치는지 확인하지 않고 있습니다. 이로 인해 중복된 예약이 발생할 수 있습니다. 업데이트 시에도 중복 예약에 대한 검증 로직을 추가하여 시간 충돌을 방지...
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
_:warning: Potential issue_ **startAt과 endAt의 순서 검증 필요** 예약 생성 시 `startAt`이 `endAt`보다 이전인지 확인하지 않고 있습니다. 시작 시간이 종료 시간보다 이후일 경우 논리적으로 맞지 않은 예약이 생성될 수 있습니다. 시작 시간이 종료 시간보다 이전인지 검증하여 잘못된 입력을 방지해야 합니...
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
_:warning: Potential issue_ **userId와 itemId의 유효성 검증 필요** 예약 생성 전에 `userId`와 `itemId`가 실제로 존재하는지 확인하지 않고 있습니다. 존재하지 않는 사용자나 아이템에 대한 예약이 생성되지 않도록 데이터베이스에서 해당 ID의 존재 여부를 확인하는 검증이 필요합니다. 다음과 같이 `u...
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
_:warning: Potential issue_ **입력값 검증 및 데이터 정제 로직 추가 필요** 현재 구현에서는 기본적인 필수 필드 검증만 수행하고 있습니다. 데이터의 무결성을 보장하기 위해 추가적인 검증이 필요합니다. 다음 사항들을 고려해주세요: ```diff export const createRoom = async (req: Requ...
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
_:warning: Potential issue_ **업데이트 데이터 검증 및 보안 강화 필요** 현재 구현에서는 업데이트 요청의 데이터 유효성 검증이 누락되어 있으며, req.body를 직접 사용하는 것은 보안상 위험할 수 있습니다. 다음과 같이 수정하는 것을 제안합니다: ```diff export const updateRoom = asyn...
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
_:warning: Potential issue_ **삭제 전 연관 데이터 확인 및 오류 처리 필요** 회의실 삭제 시 해당 회의실과 연관된 예약 정보 확인이 필요하며, 데이터베이스 작업에 대한 예외 처리가 필요합니다. 다음과 같이 수정하는 것을 제안합니다: ```diff export const deleteRoom = async (req: R...
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
_:warning: Potential issue_ **데이터베이스 오류 처리 및 페이지네이션 구현 필요** 데이터베이스 작업 시 발생할 수 있는 예외 상황에 대한 처리가 없으며, 대량의 데이터를 처리할 때 성능 문제가 발생할 수 있습니다. 다음과 같이 수정하는 것을 제안합니다: ```diff export const getAllRooms = a...
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
_:hammer_and_wrench: Refactor suggestion_ **매직 넘버를 상수로 추출하고 시간 계산 로직을 개선해야 합니다** 시간 버퍼 계산에서 매직 넘버를 사용하고 있습니다. 가독성과 유지보수성을 위해 다음과 같이 개선하는 것을 추천드립니다: ```diff +const TEN_MINUTES_IN_MS = 10 * 60 * 1...
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
_:bulb: Codebase verification_ **타임존 처리 관련 잠재적 문제점이 발견되었습니다.** `getStartAndEndOfDay` 함수는 UTC 기준으로 시간을 설정하고 있어, 다음과 같은 문제가 있습니다: - 예약 컨트롤러에서 `new Date()`로 생성된 로컬 시간이 UTC로 변환되어 의도하지 않은 날짜 범위가 설정될 ...
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
_:hammer_and_wrench: Refactor suggestion_ **함수의 견고성과 재사용성을 개선해보세요.** 다음과 같은 개선사항을 고려해보시기 바랍니다: 1. 유효하지 않은 Date 객체에 대한 처리 2. 시간 간격을 매개변수로 받아 더 유연하게 사용할 수 있도록 수정 다음과 같이 수정하는 것을 제안드립니다: ```diff -...
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
_:warning: Potential issue_ **입력값 유효성 검사가 필요합니다.** null이나 undefined가 전달될 경우 처리가 필요합니다. 다음과 같이 수정하는 것을 제안합니다: ```diff -export const getStartAndEndOfDay = (date: Date): { startOfDay: Date; endOfD...
coderabbitai[bot] created a review on a pull request on codeit-internship-group-b/codeit-resources
**Actionable comments posted: 11** <details> <summary>🧹 Outside diff range and nitpick comments (3)</summary><blockquote> <details> <summary>apps/api/src/utils/getStartAndEndOfDay.ts (1)</summary...
coderabbitai[bot] created a review on a pull request on codeit-internship-group-b/codeit-resources
**Actionable comments posted: 11** <details> <summary>🧹 Outside diff range and nitpick comments (3)</summary><blockquote> <details> <summary>apps/api/src/utils/getStartAndEndOfDay.ts (1)</summary...
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
@AdamSeungheonShin, 죄송합니다. 제가 착각했습니다. `mongoose.Types`에는 `Document` 타입이 없네요. Mongoose v7 이상에서는 `Document` 대신 `HydratedDocument`를 사용하는 것이 권장됩니다. 따라서 코드를 다음과 같이 수정하시는 것을 제안드립니다: ```typescript impor...
AdamSeungheonShin created a review comment on a pull request on codeit-internship-group-b/codeit-resources
Types 하위에 Document타입이 없던데.. 확인해봐
AdamSeungheonShin pushed 8 commits to 42-be-feat-회의실-엔드포인트-구현 codeit-internship-group-b/codeit-resources
- BE-Refactor: reservation, room controller에 데이터 유효성 검증 부분 추가 182c944
- [Refactor] userRoutes => getUsers, createUser 구현 (#73) * BE-Feat: errorHandler 추가 * BE-Feat: express asyncHandler... 73feaf0
- BE-Refactor: reservation model에 시간데이터 유효성 검증 추가 7d2fac2
- BE-Merge: update develop for push 67373ef
- BE-Refactor: 날짜 형식 검사 유틸 정규식 개선 12b68d4
- BE-Refactor: reservation 모델 인덱스 추가, 타입별 조회 status 쿼리 옵션 추가 21e84a6
- BE-Refactor: reservation model startAt, endAt 검사 로직 추가 44bf6db
- BE-Refactor: 오늘 날짜 범위 계산 로직 유틸함수로 분리, 원본데이터 변조 예방을 위한 로직 개선 d08c2f0
AdamSeungheonShin created a review comment on a pull request on codeit-internship-group-b/codeit-resources
정규식을 캡쳐링 방식으로 작성하는 이유가 있어?
bokeeeey closed an issue on codeit-internship-group-b/codeit-resources
[BE-Refactor] authRoute 수정
## 🔨 작업 내용 - 로그인 구현 - 멤버 추가 구현 - 이외 user 관련 로직 수정 ## 🖼️ 스크린샷 (선택)
bokeeeey pushed 1 commit to develop codeit-internship-group-b/codeit-resources
- [Refactor] userRoutes => getUsers, createUser 구현 (#73) * BE-Feat: errorHandler 추가 * BE-Feat: express asyncHandler... 73feaf0
dudwns0213 created a review on a pull request on codeit-internship-group-b/codeit-resources
우와 최고다 멋지다 잘생겼다 와~~~
bokeeeey pushed 38 commits to 62-be-refactor-authroute-수정 codeit-internship-group-b/codeit-resources
- FE-Setting: output export 설정 추가 e91f4f4
- FE-Merge: branch 'develop' into 28-fe-feat-공통컴포넌트-modal-제작 ffee89e
- FE-Merge: branch 'develop' into 28-fe-feat-공통컴포넌트-modal-제작 66971fb
- FE-Feat: Modal Type 적용 37f858f
- FE-Setting: web framer motion 설치 0f088a1
- FE-Setting: ui framer motion, react-dom 설치 552a3be
- FE-Setting: ui eslint react-internal으로 변경 6fcf4c1
- FE-Feat: Modal component 구현 1c82137
- FE-Feat: Modal ui index에서 export 37126e9
- FE-Feat: next config output export에 따른 use client 적용 8fc8c22
- FE-Fix: build error 해결 18b2565
- FE-Feat: client error로 인한 Modal 사용 페이지 변경 1f34ad0
- FE-Style: css 시안에 맞게 수정 및 providers test 2d908f4
- FE-Setting: jest.config.cjs로 변경 7c93e0e
- FE-Merge: branch 'develop' into 28-fe-feat-공통컴포넌트-modal-제작 a46856c
- FE-Chore: postcss.config cjs에서 js로 변경 a5f0946
- FE-Feat: Trigger useEffect 배열 최적화 2be2b1f
- FE-Feat: className 추가, 이벤트 타입 컨벤션, 키보드 핸들러 생성 bb9dae3
- FE-Feat: Root keydown escape로 수정, eslint 버그 수정 695ce2b
- FE-Remove: providers test code delete 4f953f6
- and 18 more ...
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
_:hammer_and_wrench: Refactor suggestion_ **이메일 형식 유효성 검사를 추가하세요** 현재 `email` 필드는 최소 길이와 고유성만 검사하고 있습니다. 이메일 주소의 형식이 올바른지 확인하기 위해 정규식 검증을 추가하는 것이 좋습니다. 다음과 같이 수정해 보세요: ```diff email: { type: ...
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
_:hammer_and_wrench: Refactor suggestion_ **`teams` 필드를 다른 모델과의 참조로 변경을 고려하세요** `teams` 필드가 문자열 배열로 되어 있습니다. 팀 정보를 다른 컬렉션에서 참조하려면 `Schema.Types.ObjectId`를 사용하고 `ref` 옵션으로 `Team` 모델을 지정하는 것이 좋습니다....
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
_:warning: Potential issue_ **프로필 이미지 처리 로직 개선 필요** 타입 캐스팅을 사용한 현재 방식은 `req.file`이 `undefined`일 때 런타임 에러가 발생할 수 있습니다. 다음과 같이 안전한 방식으로 수정하는 것을 추천드립니다: ```diff - const profileImageUrl = (req.fi...
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
_:hammer_and_wrench: Refactor suggestion_ **파일 타입 처리 개선 필요** `CreateUserRequest` 인터페이스의 `file` 속성에서 유니온 타입을 사용하고 있습니다. S3를 사용하는 환경에서는 `Express.MulterS3.File` 타입만 필요할 것으로 보입니다. 다음과 같이 수정하는 것을 추천...
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
_:hammer_and_wrench: Refactor suggestion_ **이름 필드의 최대 길이를 재고해 보세요** `name` 필드의 최대 길이가 10자로 제한되어 있습니다. 이는 일부 사용자에게 너무 짧을 수 있습니다. 최대 길이를 늘리거나 제한을 없애는 것을 고려해 보세요. <!-- This is an auto-generated com...
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
_:hammer_and_wrench: Refactor suggestion_ **불필요한 배열 생성 제거** `newTeams` 배열을 생성하고 spread 연산자로 복사하는 과정이 불필요합니다. 다음과 같이 단순화할 수 있습니다: ```diff - const newTeams: string[] = []; - - if (teams) newTe...