[refactor] 프로필 편집화면의 이름, 소개 수정 화면 분리 / 프로필 이미지 변경시 이미지 1mb 초과시 압축 #138
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
JongHoooon
commented
Jul 12, 2023
•
JongHoooon
commented
- [Refactor] 프로필 이미지 변경시 이미지 사이즈 제한 필요 #126
- [refactor] 프로필 편집화면의 이름, 소개 수정 화면 분리 #137
Comment on lines
118
to
120
|
|
||
| var pushNameVC: PublishRelay<#NameVC> = .init() | ||
| var pushNameVC: PublishRelay<ValidateNameVC> = .init() | ||
| } |
There was a problem hiding this comment.
Relay 타입의 프로퍼티 이름을 지을 때 규칙같은게 있으신가요? 이름만 봐서는 relay 타입이라 생각하기는 어려워서요
There was a problem hiding this comment.
- https://ios-development.tistory.com/707?category=1008271
- https://github.com/kudoleh/iOS-Clean-Architecture-MVVM/blob/master/ExampleMVVM/Presentation/MoviesScene/MoviesList/ViewModel/MoviesListViewModel.swift
제가 참고했던 자료들인데 output protocol로 작성하는 부분 보면 딱히 relay명시는 안해주고 있어요
저도 중간에 relay 타입이라고 생각하기 힘들다고 느꼈어서 중간에 1, 2개 정도는 끝에 ooooRealy 이런식으로 작성해본거 있어요
앞으로 relay, observable 같은거 선언할 때 끝에 ooooRelay, ooooObservable 이런식으로 명시해주도록 할까요??
There was a problem hiding this comment.
넵 그래 주시면 감사하죠! 이슈 수정하면서 해당 파일에 relay 같은 타입 있으면 틈틈히 바꿔주시면 좋을 것 같습니다.
There was a problem hiding this comment.
개인적인 생각으로는 프로퍼티 네임만 봐도 어떤 앤지를 알 수 있었으면 해서요!
seungwon-yoo
approved these changes
Jul 15, 2023
- 소개 작성 화면 분리 - 프로필 이미지 변경 로직 수정 - 이미지가 수정된 상태에서 나갈때 alert 추가
name값을 전달하지 않으면 nil 값이 전달돼 user name이 nil이 되는 버그 수정
JongHoooon
force-pushed
the
refactor/editProfile
branch
from
68a7365
to
bb3791a
Compare
This was referenced Jul 15, 2023
JongHoooon
added a commit
that referenced
this pull request
Jul 17, 2023
* [refactor] 닉네임 수정 화면 분리 * [refactor] 소개 작성화면 분리, 프로필 이미지 변경 로직 수정 - 소개 작성 화면 분리 - 프로필 이미지 변경 로직 수정 - 이미지가 수정된 상태에서 나갈때 alert 추가 * [refactor] 프로필 사진 변경시 1mb 초과하면 압축 * [Fix] 기존 name과 같은 값으로 요청해도 name 값 전달하게 수정 name값을 전달하지 않으면 nil 값이 전달돼 user name이 nil이 되는 버그 수정 * [Refactor] relay 프로퍼티들 명시
JongHoooon
added a commit
that referenced
this pull request
Jul 17, 2023
* [refactor] 닉네임 수정 화면 분리 * [refactor] 소개 작성화면 분리, 프로필 이미지 변경 로직 수정 - 소개 작성 화면 분리 - 프로필 이미지 변경 로직 수정 - 이미지가 수정된 상태에서 나갈때 alert 추가 * [refactor] 프로필 사진 변경시 1mb 초과하면 압축 * [Fix] 기존 name과 같은 값으로 요청해도 name 값 전달하게 수정 name값을 전달하지 않으면 nil 값이 전달돼 user name이 nil이 되는 버그 수정 * [Refactor] relay 프로퍼티들 명시
JongHoooon
added a commit
that referenced
this pull request
Jul 17, 2023
* [refactor] 닉네임 수정 화면 분리 * [refactor] 소개 작성화면 분리, 프로필 이미지 변경 로직 수정 - 소개 작성 화면 분리 - 프로필 이미지 변경 로직 수정 - 이미지가 수정된 상태에서 나갈때 alert 추가 * [refactor] 프로필 사진 변경시 1mb 초과하면 압축 * [Fix] 기존 name과 같은 값으로 요청해도 name 값 전달하게 수정 name값을 전달하지 않으면 nil 값이 전달돼 user name이 nil이 되는 버그 수정 * [Refactor] relay 프로퍼티들 명시
# for free
to join this conversation on GitHub.
Already have an account?
# to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.