Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
Leekee0905 pushed 21 commits to feat/card-scroll nninyeong/challenge-b
- fix: 로그인 여부 확인용 메서드 변경 54ad8f1
- feat: 이메일 회원가입 기능 추가 ad7e12e
- add: 리뷰 캐러셀 생성 863d220
- feat: 리뷰 캐러셀 구현 3b9dcec
- feat: 이메일 로그인, zod 유효성검사 추가 0ce6342
- refactor: 입력 폼 부분 컴포넌트 분리 8983bb1
- style: 에러 메시지 출력 시 레이아웃 깨지지 않도록 flex -> grid 변경 c009c67
- Merge branch 'develop' of https://github.com/nninyeong/challenge-b into feat/social-login c355973
- fix: 리뷰 캐러셀 화면 비율 조정 ffaf07a
- Merge branch 'develop' of https://github.com/nninyeong/challenge-b into feat/carousel 9b37d9a
- feat: reviews 데이터 연결 6939456
- add: ReviewsCarousel 컴포넌트 추가 a710d84
- bugfix: 이메일 로그인하는 경우에도 헤더의 네비게이션 상태가 바로 변경될 수 있도록 수정 453c1c1
- remove: 사용하지 않는 변수 제거 d2cf72c
- fix: 주석 제거 18579ce
- fix: 캐러셀 리뷰 수정 43f2e7e
- feat:무한스크롤기능구현, 후기페이지컴포넌트분리, 기본 레이아웃작성 342a162
- Merge pull request #10 from nninyeong/feat/social-login Feat/social login: 이메일 회원가입/로그인 기능 추가 72ebe73
- Merge pull request #11 from nninyeong/feat/carousel Feat/carousel 2666f12
- Merge pull request #12 from nninyeong/feat/infiniteScrollReview feat:무한스크롤기능구현, 후기페이지컴포넌트분리, 기본 레이아웃작성 a6a2c66
- and 1 more ...
nninyeong created a review comment on a pull request on nninyeong/challenge-b
console.log 지워주시면 좋을 것 같아요 👍👍👍👍
nninyeong created a review comment on a pull request on nninyeong/challenge-b
나중에 여유가 된다면!! 저희 database.types.ts 가져온 거 기반으로 수빈님이 설정해두신 타입 파일 있는데 거기 Review 타입이랑 통일하면 좋을 것 같아요! reviewData.types.ts에 있습니다!
Leekee0905 pushed 2 commits to develop nninyeong/challenge-b
Leekee0905 closed a pull request on nninyeong/challenge-b
feat:무한스크롤기능구현, 후기페이지컴포넌트분리, 기본 레이아웃작성
## ✅ 작업 사항 -후기 페이지 무한스크롤 기능구현 - 기본 레이아웃 작성 ## 📸 스크린샷  <br/> ## 🧑💻 기타 - 사진만 보이는 부...
Leekee0905 created a review comment on a pull request on nninyeong/challenge-b
next.js에서 try/catch문은 server action에서는 최대한 피하는 것을 추천하고 있어서 이 부분도 나중에 같이 리팩토링 하면 좋을 것 같습니다!
Leekee0905 created a review comment on a pull request on nninyeong/challenge-b
나중에 같이 query hook으로 빼는거 해보면 좋을 것 같아요!
nninyeong created a review comment on a pull request on nninyeong/challenge-b
저희 console.log 대신 console.error만 남겨두기로 해서 이것만 변경해주시면 감사하겠습니당!!
nninyeong created a review comment on a pull request on nninyeong/challenge-b
혹시 나중에 프리패치 적용하신다면..!!! 여기서 프리패치해도 될 것 같아요!
Leekee0905 pushed 9 commits to develop nninyeong/challenge-b
- add: 리뷰 캐러셀 생성 863d220
- feat: 리뷰 캐러셀 구현 3b9dcec
- fix: 리뷰 캐러셀 화면 비율 조정 ffaf07a
- Merge branch 'develop' of https://github.com/nninyeong/challenge-b into feat/carousel 9b37d9a
- feat: reviews 데이터 연결 6939456
- add: ReviewsCarousel 컴포넌트 추가 a710d84
- fix: 주석 제거 18579ce
- fix: 캐러셀 리뷰 수정 43f2e7e
- Merge pull request #11 from nninyeong/feat/carousel Feat/carousel 2666f12
Leekee0905 closed a pull request on nninyeong/challenge-b
Feat/carousel
## ✅ 작업 사항 - 화면 비율에 맞춰 리뷰 캐러셀 구현 - reviews 데이터 연결 ## 📸 스크린샷  <br/> ## 🧑...
Leekee0905 created a review comment on a pull request on nninyeong/challenge-b
서버액션으로 따로 빼도 좋을 것 같아요!
Leekee0905 pushed 9 commits to develop nninyeong/challenge-b
- fix: 로그인 여부 확인용 메서드 변경 54ad8f1
- feat: 이메일 회원가입 기능 추가 ad7e12e
- feat: 이메일 로그인, zod 유효성검사 추가 0ce6342
- refactor: 입력 폼 부분 컴포넌트 분리 8983bb1
- style: 에러 메시지 출력 시 레이아웃 깨지지 않도록 flex -> grid 변경 c009c67
- Merge branch 'develop' of https://github.com/nninyeong/challenge-b into feat/social-login c355973
- bugfix: 이메일 로그인하는 경우에도 헤더의 네비게이션 상태가 바로 변경될 수 있도록 수정 453c1c1
- remove: 사용하지 않는 변수 제거 d2cf72c
- Merge pull request #10 from nninyeong/feat/social-login Feat/social login: 이메일 회원가입/로그인 기능 추가 72ebe73
Leekee0905 closed a pull request on nninyeong/challenge-b
Feat/social login: 이메일 회원가입/로그인 기능 추가
## ✅ 작업 사항 - 회원가입, 로그인 입력 폼 유효성검사 추가 (이메일 형식 지켰는지, 비밀번호끼리 일치하는지, 비밀번호 4자 이상, 이름 2자이상) - 이메일 회원가입/로그인 기능 추가 - 헤더에 인증상태 바로 바반영할 수 있도록 함 ## 📸 스크린샷 ![이메일회원가입](https://github.com/us...
Leekee0905 created a review comment on a pull request on nninyeong/challenge-b
props 타입은 제일 위에서 선언해주는게 좋을 것 같다는 생각입니다