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_ **`id` 필드의 필수성 검토가 필요합니다.** `id` 필드가 선택적(`?`)으로 정의되어 있습니다. 회원 데이터의 고유 식별자로서 이는 필수값이어야 합니다. 다음과 같이 수정을 제안드립니다: ```diff - id?: string | number; + id: string | number...
coderabbitai[bot] created a review on a pull request on codeit-internship-group-b/codeit-resources
**Actionable comments posted: 3** <details> <summary>🧹 Outside diff range and nitpick comments (1)</summary><blockquote> <details> <summary>packages/ui/src/components/common/Dropdown/index.tsx (1...
coderabbitai[bot] created a review on a pull request on codeit-internship-group-b/codeit-resources
**Actionable comments posted: 3** <details> <summary>🧹 Outside diff range and nitpick comments (1)</summary><blockquote> <details> <summary>packages/ui/src/components/common/Dropdown/index.tsx (1...
miraclee1226 pushed 4 commits to 69-fe-feat-멤버-관리-페이지-ui-구현 codeit-internship-group-b/codeit-resources
cindycho0423 pushed 7 commits to 59-fe-feat-좌석예약페이지-ui-제작 codeit-internship-group-b/codeit-resources
AdamSeungheonShin pushed 4 commits to 39-be-feat-좌석-엔드포인트-구현 codeit-internship-group-b/codeit-resources
bokeeeey pushed 5 commits to 62-be-refactor-authroute-수정 codeit-internship-group-b/codeit-resources
- [Refactor] userRoutes => getUsers, createUser 구현 (#73) * BE-Feat: errorHandler 추가 * BE-Feat: express asyncHandler... 73feaf0
- [Feat] 회의실 엔드포인트 구현 (#75) * BE-Refactor: 필드명 호환성을 위해 각 필드 id명 _id로 변경 * BE-Feat: reservation 모델, 스키마 정의 * BE-F... 90bff05
- [Fix] 헤더 요구사항에 맞게 날짜 보여주는 함수 수정 (#76) FE-Fix: 헤더 요구사항에 맞게 날짜 보여주는 함수 수정 4d8920c
- BE-Chore: 스프레드문법 Error로 인한 dev pull 진행용 commit db53fcb
- BE-Merge: develop pull & merge 진행 6ff072f
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
@bokeeeey, 알겠습니다. 감사합니다. (｡•́︿•̀｡) <!-- This is an auto-generated reply by CodeRabbit -->
bokeeeey created a review comment 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
@bokeeeey, 죄송합니다. 더 친절하게 답변 드리도록 하겠습니다. （；＿；） <!-- This is an auto-generated reply by CodeRabbit -->
bokeeeey created a review comment on a pull request on codeit-internship-group-b/codeit-resources
말투 마음에 안들어 -1점
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
_:hammer_and_wrench: Refactor suggestion_ **불필요한 타이머 사용을 검토해 주세요.** 폼 데이터 초기화에 타이머를 사용하는 것이 불필요해 보입니다. 패널이 닫힐 때 즉시 초기화해도 무방할 것 같습니다. ```diff useEffect(() => { if (!isOpen) { - const timer...
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
_:warning: Potential issue_ **모달의 상태 처리가 필요합니다.** 탈퇴 처리 중 로딩 상태와 에러 상태에 대한 처리가 필요합니다: - 탈퇴 처리 중 로딩 인디케이터 표시 - 에러 발생 시 사용자에게 알림 - 탈퇴 버튼 비활성화 처리 <!-- This is an auto-generated comment by CodeRabbi...
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
_:warning: Potential issue_ **접근성 개선이 필요합니다.** 다음과 같은 접근성 관련 속성들을 추가하는 것이 좋습니다: - 모달에 `aria-modal="true"` - 닫기 버튼에 `aria-label` - 폼 필드에 `aria-required` 또한, CSS 클래스에서 하드코딩된 값들을 변수로 분리하는 것이 좋습니다:...
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
_:warning: Potential issue_ **폼 제출 로직 구현이 필요합니다.** 현재 TODO 주석으로 표시된 폼 제출 로직에 다음 사항들을 추가하는 것이 좋습니다: - 폼 유효성 검사 - 에러 처리 - 서버 통신 상태 관리 폼 제출 로직 구현에 도움이 필요하시다면 말씀해 주세요. <!-- This is an auto-generat...
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
_:hammer_and_wrench: Refactor suggestion_ **접근성과 사용자 경험을 개선할 수 있습니다.** 컴포넌트의 접근성과 사용자 경험을 높이기 위한 몇 가지 제안사항이 있습니다. 1. 역할 선택 드롭다운에 대한 접근성 레이블 추가: ```diff - <div data-dropdown="true"> + <...
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
_:hammer_and_wrench: Refactor suggestion_ **이벤트 핸들러 개선 제안** 현재 이벤트 핸들러들이 기본적인 기능은 잘 수행하고 있지만, 다음과 같은 개선사항을 제안드립니다: 1. 에러 처리 로직 추가 2. 더 구체적인 TypeScript 이벤트 타입 사용 ```typescript const handleMembe...
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
_:hammer_and_wrench: Refactor suggestion_ **`id` 필드의 타입 정의를 개선해 주세요.** `id` 필드가 선택적(optional)으로 정의되어 있는데, 회원 데이터의 고유 식별자로서 필수값으로 지정하는 것이 더 안전할 것 같습니다. 또한 `string` 타입 대신 더 구체적인 타입을 사용하면 좋을 것 같습니다....
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
_:hammer_and_wrench: Refactor suggestion_ **`teams`와 `role` 필드의 타입을 더 구체화해 주세요.** `teams`와 `role` 필드가 현재 너무 광범위하게 정의되어 있습니다. 가능한 값들을 열거형(enum)으로 제한하면 타입 안정성을 높일 수 있습니다. 다음과 같은 개선을 제안드립니다: ```ty...
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
_:hammer_and_wrench: Refactor suggestion_ **인터페이스 간 일관성 검토가 필요합니다.** `MemberWithStaticImport`와 `MemberWithStaticImage` 인터페이스가 `profileImage` 필드를 제외하고 동일한 구조를 가지고 있습니다. 중복을 줄이고 유지보수성을 높이기 위해 공통 필드...
coderabbitai[bot] created a review on a pull request on codeit-internship-group-b/codeit-resources
**Actionable comments posted: 9** <details> <summary>🧹 Outside diff range and nitpick comments (3)</summary><blockquote> <details> <summary>packages/ui/src/utils/constants/image.ts (2)</summary><...
coderabbitai[bot] created a review on a pull request on codeit-internship-group-b/codeit-resources
**Actionable comments posted: 9** <details> <summary>🧹 Outside diff range and nitpick comments (3)</summary><blockquote> <details> <summary>packages/ui/src/utils/constants/image.ts (2)</summary><...
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
@bokeeeey, 동의해 주셔서 감사합니다! merge할 때 console.log를 삭제하면 코드의 품질이 향상될 것입니다. 또한, `react/no-unescaped-entities` 규칙을 준수하여 따옴표를 HTML 엔티티로 변경하고, `@typescript-eslint/no-confusing-void-expression` 규칙에 따라 void ...
coderabbitai[bot] created a review comment on a pull request on codeit-internship-group-b/codeit-resources
@bokeeeey, `file`은 이미 `e.target.files?.[0]`로 단일 파일을 가져온 것이므로 `file` 자체가 파일 객체입니다. 따라서 `file[0]`은 존재하지 않습니다. `setFormData`에서는 `file`을 그대로 사용하시면 됩니다. <!-- This is an auto-generated reply by CodeRabb...