search

doyeonjeong / Tublock

Just YouTube Blocking App
1 stars 0 forks source link

✨[Feat]: SetMessageView Localizing 적용, 고도화 작업 #45

Closed kangddong closed 1 year ago

kangddong commented 1 year ago

변경사항

  1. Toast 라이브러리 추가
  2. 로컬라이징 적용
  3. 85자 제한 기능 추가
  4. 85자 제한 시 토스트 기능 추가

스크린샷 2023-05-01 오전 2 12 40

doyeonjeong commented 1 year ago

엥 이거 컨플릭트 수정도 본인이 해야하는거였어요? 내가 작업한거랑 겹쳐서 제가 처리했는데 최종 작업자가 나라서 어프로브를 할 수 없다고 뜨네요 .. (입술꽉)

스크린샷 2023-05-02 오후 1 09 11
kangddong commented 1 year ago

아렉스는 이 이슈를 잊지않을 것입니다

doyeonjeong commented 1 year ago

다른 사람이 Conflict 수정한 후 Merge 불가능했던 문제

스크린샷 2023-05-02 오후 4 19 33

브랜치 룰에 Require approval of the most recent reviewable push가 걸려있어서 발생했던 오류

아렉스의 PR을 맨 마지막으로 Conflict 수정하여 Merge 한 사람이 나라서 아렉스가 본인 PR을 리뷰할수도 없는 상황이고 팀원이 2명이라 다른 사람에게 코드 리뷰를 부탁하기도 애매한 상황

어쩔 수 없이 브랜치 룰을 잠시 풀고 Merge 하는 방법으로 해결했다.