Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
Kiyoung-Kim-57 created a review comment on a pull request on boostcampwm-2024/iOS04-PhotoGether
- 상수의 경우에 따로 빼서 사용할 경우 휴먼 에러를 방지할 수 있습니다! 지금은 크게 안 와닿으실 수 있지만 동일한 상수를 여러 곳에서 사용하게 될 경우 쉽게 휴먼 에러가 방지됩니다!
Kiyoung-Kim-57 created a review comment on a pull request on boostcampwm-2024/iOS04-PhotoGether
지금은 publisher에서 filter를 걸어서 구분하려고 하고 있습니다!!
0Hooni created a review comment on a pull request on boostcampwm-2024/iOS04-PhotoGether
이렇게 constants를 따로 빼두는게 영규님과 기영님 스타일이신것 같아요! 저는 뭔가 디자인 요소를 검토해볼때 피그마와 비교하면서 확인을 한번씩 해보는데 검토 기준으로는 snp에 바로 값이 써져있지 않아서 검토할 때는 불편하긴 한더라고요 🥲 프로그래밍 기준으로는 어떤 장점을 느꼈는지 궁금합니다!! ☺️
0Hooni created a review comment on a pull request on boostcampwm-2024/iOS04-PhotoGether
피그마 기준으로 오브젝트 이름이 `Ellipse`로 되어있습니다!! 오타인것 같아요 ☺️ 아니면 혹시 제가 모르는 의도가 있는건가요 😮
0Hooni created a review comment on a pull request on boostcampwm-2024/iOS04-PhotoGether
이런식으로 호스트와 게스트에게 액션을 나눌수도 있겠네요!!! 저희는 Host, Guest용 VC를 따로 두고 있는데 뭔가 통합도 잘 생각해보면 될 수 있겠다는 순간이었습니다 👍
0Hooni created a review comment on a pull request on boostcampwm-2024/iOS04-PhotoGether
제가 너무 멍청한건지 지금 이부분 어떤식으로 동적으로 작동되고있는지 잘 그려지지가 않아서 나중에 설명 한번만 해주시면 정말 감사하겠습니다 🙇♂️
0Hooni created a review comment on a pull request on boostcampwm-2024/iOS04-PhotoGether
연속 터치 방지 코드인가 봅니다!!! 👍
0Hooni created a review comment on a pull request on boostcampwm-2024/iOS04-PhotoGether
이렇게 해서 닉네임라벨 아래 비디오뷰가 가나보네요 😮 addSubView에서 순서로 정하는 방식이 아니라서 어디에 위치될것이라는 직관적인 느낌도 들고 좋았습니다 👍 다만 반대로 `addViews()`와 `setConstraints()`에서 처리될 작업들이 따로 나와있다는 느낌을 받기도 하는데 이렇게 View 자체를 weak로 두고, 외...
0Hooni created a review comment on a pull request on boostcampwm-2024/iOS04-PhotoGether
fatalError처럼 사용시 터뜨리는걸 넘어서 아예 호출을 막는 방식인가 보네요!! 영규님 코드 매번 신기한것들이 많아서 재밌습니다 👍 배워갑니다 ☺️
0Hooni created a review on a pull request on boostcampwm-2024/iOS04-PhotoGether
**LGTM** 💯 몇가지 궁금한 점들을 적어놨는데 나중에 코멘트 한번만 부탁드립니다 🫡 수고 많으셨습니다!
0Hooni created a review on a pull request on boostcampwm-2024/iOS04-PhotoGether
**LGTM** 💯 몇가지 궁금한 점들을 적어놨는데 나중에 코멘트 한번만 부탁드립니다 🫡 수고 많으셨습니다!
Kiyoung-Kim-57 created a review comment on a pull request on boostcampwm-2024/iOS04-PhotoGether
[aca69eb](https://github.com/boostcampwm-2024/iOS04-PhotoGether/pull/58/commits/aca69eba34fba3fd89404c9582308f8fa27ca583) 수정했습니다!
Kiyoung-Kim-57 created a review comment on a pull request on boostcampwm-2024/iOS04-PhotoGether
[aca69eb](https://github.com/boostcampwm-2024/iOS04-PhotoGether/pull/58/commits/aca69eba34fba3fd89404c9582308f8fa27ca583) 수정했습니다!
Kiyoung-Kim-57 created a review comment on a pull request on boostcampwm-2024/iOS04-PhotoGether
[aca69eb](https://github.com/boostcampwm-2024/iOS04-PhotoGether/pull/58/commits/aca69eba34fba3fd89404c9582308f8fa27ca583) 이런 실수를....위의 커멘트 내용 전부 반영하여 수정했습니다!
Kiyoung-Kim-57 pushed 1 commit to feat/#44-host-photo-room-ui boostcampwm-2024/iOS04-PhotoGether
- refactor/#44 :: 상속되지 않는 class에 final 적용 aca69eb
Kiyoung-Kim-57 created a review comment on a pull request on boostcampwm-2024/iOS04-PhotoGether
- 우선 bindInput 같은 경우는 filter를 이용해서 게스트의 불필요한 바인딩을 방지할 것 같습니다! - bindOutput의 경우에는 타이머 output이 호스트 게스트 결국 둘 다 필요한 것이라 현재는 따로 filter를 적용하지 않았으나 추후 호스트와 게스트의 분리가 필요한 바인딩의 경우 filter로 구분할 예정입니다! - Vie...
0Hooni opened a pull request on boostcampwm-2024/iOS04-PhotoGether
[FEAT/#53] 스티커를 추가 기능 및 테스트 코드 구현
## 🤔 배경 - 동시 편집을 위한 더미 데이터로 스티커가 필요했다. - 해당 더미데이터를 편집 화면에 놓을 수 있어야 됐다. ## 📃 작업 내역 - Sticker 더미 데이터를 추가했습니다. - Sticker 버튼을 눌러 랜던한 스티커를 편집 화면 위에 놓을 수 있습니다. - 또한 매번 사용자 기준 같은 크기의 스티거가 같은 위치에 추...
hsw1920 created a review comment on a pull request on boostcampwm-2024/iOS04-PhotoGether
아 기본 닉네임이 있었군여! 넵
hsw1920 created a review comment on a pull request on boostcampwm-2024/iOS04-PhotoGether
넵 그 방향으로 생각하겠습니다!
youn9k created a review comment on a pull request on boostcampwm-2024/iOS04-PhotoGether
ConnectionClient를 Data 레이어로 옮기고 딕셔너리나 배열로 관리하는 ConnectionRepository를 생각중입니다. ViewModel에서 상태와 로직을 직접 담당하지 않고 로직은 UseCase에게, 상태관리는 ConnectionRepository가 담당하면 역할 분리가 확실히되어 좋을 것 같단 생각입니다!
youn9k created a review comment on a pull request on boostcampwm-2024/iOS04-PhotoGether
바뀔 수도 있는데 우선은 공통으로 사용하는 방향으로 생각중입니다!