Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
Largopie pushed 24 commits to feat/394-go-back woowacourse-teams/2024-momo
- [BE] 프로덕션 서버 무중단 배포 환경 구축 (#413) * chore: 서브모듈 업데이트 반영 - 무중단 배포 스크립트 추가 * refactor(backend-prod-cd): 배포과정에서 무중... 3120495
- [FE] 약속 생성 확인 모달 복구 develop 브랜치에 적용 (#415) * feat(MeetingCreationConfirmModal): 약속 생성 확인 모달 구현 * feat(CreateMeeti... b70fad0
- design: 토스트 컨테이너 패딩 조절 - 토스트 메시지의 padding 속성으로 입력부분이 가려지는 문제가 있어 위와 아래의 패딩을 0으로 설정 51df7fc
- refactor: GlobalLayout 폴더 구조 수정 28b28d1
- chore: 라이브러리 수정 - @types/jest 설치 - storybook 업데이트 e093d5e
- feat(ContentLayout): 전역적으로 사용할 main태그 레이아웃 구현 2d4064b
- refactor(Header): 로그인 버튼 제거 및 추상화 48b610b
- feat(useRouter): 라우팅 커스텀 훅 구현 e67aa6c
- chore: 뒤로가기, 공유 SVG 추가 및 체크 SVG 수정 152c30e
- refactor: 페이지별 헤더 추가, ContentLayout 적용 - 필요한 헤더에 뒤로가기 라우팅, 공유 기능 구현 3414ca4
- design: 전체적인 레이아웃 변경으로 인한 CSS 수정 d901b4f
- refactor: 새로고침 시, failed to fetch 에러가 발생하는 문제로 기본 형태를 반환할 수 있도록 수정 e6c4f26
- refactor(useRouter): useNavigate를 담는 변수명 수정 2bb0450
- refactor(useRouter): uuid제거 및 뒤로가기 기능 추가 03e4bf3
- feat(useUuid): uuid를 불러오는 훅 구현 08a82ad
- refactor: useParams, navigate를 사용하는 함수에 useRouter, useUuid 적용 ec759ba
- refactor(GlobalLayout, ContentLayout): `index`로 이름 수정 306d6ea
- feat(UuidLayout, UuidProvider): uuid 내부에서만 사용할 수 있는 UuidLayout 구현 b05cdeb
- refactor: useUuid 삭제 및 UuidContext로 대체 d1c16ac
- refactor(meetingHandlers): 중복 로직 `||` 연산자로 수정 7dcd6fb
- and 4 more ...
Yoonkyoungme created a review on a pull request on woowacourse-teams/2024-momo
🐫 낙타 🐪 고생하셨습니다!!!!!! 코드 만 줄의 변경이라니~~~~ 엄청나군요 의견 교환하는 과정 재미있었습니다 ㅎㅅㅎ 다음은 웹 접근성 함께 작업하러 가시죠!
hwinkr created a review on a pull request on woowacourse-teams/2024-momo
안정적인 뒤로가기 기능을 제공하기 위해서 열심히 코드를 작성하셨네요 낙타! 정말 고생하셨습니다~ 빙봉과 티키타카 코드 리뷰 정말 인상적이네요. 모모 서비스에서 중심이 되는 기능을 개발하는 PR에는 코멘트가 기본적으로 30개는 달리는 것 같네요...ㅋㅋ~ 드디어 뒤로가기 기능이 생겼다니 백엔드 모모들이 정말 좋아할 것 같네요! 고생 많...
hwinkr created a review on a pull request on woowacourse-teams/2024-momo
안정적인 뒤로가기 기능을 제공하기 위해서 열심히 코드를 작성하셨네요 낙타! 정말 고생하셨습니다~ 빙봉과 티키타카 코드 리뷰 정말 인상적이네요. 모모 서비스에서 중심이 되는 기능을 개발하는 PR에는 코멘트가 기본적으로 30개는 달리는 것 같네요...ㅋㅋ~ 드디어 뒤로가기 기능이 생겼다니 백엔드 모모들이 정말 좋아할 것 같네요! 고생 많...
Largopie created a comment on a pull request on woowacourse-teams/2024-momo
https://github.com/woowacourse-teams/2024-momo/pull/411#pullrequestreview-2376622314 @Yoonkyoungme 좋은 질문 남겨주셔서 감사합니다 :) [Q] > 경로 배열 선언 후 Header 라우팅처대한 낙타의 의견이 궁금합니다💭 변경사항에 대한 유연한 대처가 어렵다고...
Largopie pushed 1 commit to feat/394-go-back woowacourse-teams/2024-momo
- refactor(useRouter): routeWithState 함수 추가 및 useNavigate 사용처 useRouter로 변경 231eac3
Yoonkyoungme created a review comment on a pull request on woowacourse-teams/2024-momo
useRouter에서 router to 함수가 useNavigate 로 경로 이동하는 함수자나 그러면 지금 코드 내에서도 ``` const navigate = useNavigate(); <button onClick={() => navigate("/"}></button> ``` 이렇게 이동하는 방식과 낙타가 만들어준 useRouger의 route...
Largopie created a review on a pull request on woowacourse-teams/2024-momo
@hwinkr @Yoonkyoungme 리뷰 반영사항 반영하고 다시 요청합니다~! 변경된 자세한 사항은 노션 하단에 [해빙 리뷰 후 반영사항](https://paper-mass-5ff.notion.site/8e55c81023334a869c8779094ba5f595?pvs=4) 에서 확인할 수 있습니다. ## uuid 반환 로직 Conte...
Largopie created a review on a pull request on woowacourse-teams/2024-momo
@hwinkr @Yoonkyoungme 리뷰 반영사항 반영하고 다시 요청합니다~! 변경된 자세한 사항은 노션 하단에 [해빙 리뷰 후 반영사항](https://paper-mass-5ff.notion.site/8e55c81023334a869c8779094ba5f595?pvs=4) 에서 확인할 수 있습니다. ## uuid 반환 로직 Conte...
Largopie created a review on a pull request on woowacourse-teams/2024-momo
@hwinkr @Yoonkyoungme 리뷰 반영사항 반영하고 다시 요청합니다~! 변경된 자세한 사항은 노션 하단에 [해빙 리뷰 후 반영사항](https://paper-mass-5ff.notion.site/8e55c81023334a869c8779094ba5f595?pvs=4) 에서 확인할 수 있습니다. ## uuid 반환 로직 Conte...
Largopie pushed 10 commits to feat/394-go-back woowacourse-teams/2024-momo
- refactor(useRouter): useNavigate를 담는 변수명 수정 bc3a154
- refactor(useRouter): uuid제거 및 뒤로가기 기능 추가 c4fd735
- feat(useUuid): uuid를 불러오는 훅 구현 f442c22
- refactor: useParams, navigate를 사용하는 함수에 useRouter, useUuid 적용 d7d370f
- refactor(GlobalLayout, ContentLayout): `index`로 이름 수정 b4aa912
- feat(UuidLayout, UuidProvider): uuid 내부에서만 사용할 수 있는 UuidLayout 구현 49d4159
- refactor: useUuid 삭제 및 UuidContext로 대체 4ee7194
- refactor(meetingHandlers): 중복 로직 `||` 연산자로 수정 e719a7d
- feat(BackButton): 뒤로가기 버튼 컴포넌트 구현 36a056a
- refactor: 사용되던 뒤로가기 버튼을 `BackButton` 컴포넌트로 교체 223ec06
hw0603 created a review comment on a pull request on woowacourse-teams/2024-momo
음.. 별도로 레퍼런스가 있다기보다는 보통 영문에서 `lt`, `lte`, `gt`, `gte` 와 같은 표기를 많이 사용해서 그게 자연스럽다고 느꼈던 것 같아요. 자바 계열에서는 JUnit의 검증 메서드 명명법을 참고하기는 합니다. 저 메서드 보자마자 JUnit의 `greaterThanOrEqualTo()` 가 떠올랐어요ㅋㅋㅋ
seokmyungham created a review comment on a pull request on woowacourse-teams/2024-momo
비즈니스 정책에 해당한다고 생각해서 비즈니스 로직이 수행되는 위치에서 수행되어야 할 것 같아요.
ikjo39 pushed 5 commits to feat/398-recommend-min-duration woowacourse-teams/2024-momo
ikjo39 created a review comment on a pull request on woowacourse-teams/2024-momo
최소 시간 보장이 '빠르게 만나요'의 옵션으로 정책이 수정되면서 `MIN_TIME_EARLIER_ORDER`이 필요하지 않아 삭제하였습니다!
ikjo39 created a review comment on a pull request on woowacourse-teams/2024-momo
동의합니다! 재즈 리뷰에도 질문 남겼지만 그러면 controller의 parameter 단에서 validation 라이브러리를 통해 검증하는건 어떨까요?
ikjo39 created a review comment on a pull request on woowacourse-teams/2024-momo
오호 이 부분은 흥미롭네요. 일반적으로 사용되는 메서드 명명법을 참고할 만한 곳이 있으면 공유 요청 드려도 될까요? 추후 메서드 명명에 도움이 될 것 같습니다!
ikjo39 created a review comment on a pull request on woowacourse-teams/2024-momo
이 부분도 정책적인 사인이라 스크럼으로 얘기 나눴었죠. 시간 단위로 나누는 것으로 결론내었으니 반영해보도록 할게요!