-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Feat] #186 - 유저프로필 구현 하였습니다. #223
Merged
The head ref may contain hidden characters: "#186-\uC720\uC800\uD504\uB85C\uD544-\uC7AC-\uAD6C\uD604"
Merged
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
전체를 CVC에 넣기 위해 Cell 파일을 만들 었습니다.
해당 section == 3 부분만 스크롤이 되어서 유저가 보는 화면이 작게 보였는데, 이 부분을 코드 리팩터링을 하여 수정 하였습니다.
필요없는 코드 삭제, 매직 넘버 상수화 등
실시간으로 바뀌는 유저 데이터를 받아오기 위함 입니다.
삭제 하지 마십 시오..
513sojin
approved these changes
Dec 13, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in 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.
🌱 작업한 내용
🌱 PR Point
유저 프로필을 보면 셀 이 총 4개가 있습니다.
유저 이름 부분
,progress 부분
,업로드한 코스(label 부분)
,유저 코스 부분
이렇게 총 4개가 있는데
전체를 컬렉션 뷰로 하여 case 별로 할지 (-> 이렇게 되면 각각 부분을 Cell로 파일을 나눠줘야 함)
현재처럼 그냥 한 파일에 묶을지 고민입니다. -> (현재 뷰 내에 작은 CVC 존재)
Toast 메시지를 좀 더 뚜렷하게 바꾸고 싶어서 대충 만들었는데 테스트해 보다가 필요 없으면 삭제할 예정이라 extension으로 따로 빼놨습니다.
toast 메시지
총 1.2초를 띄우고 총 1.5초 뒤에 자동으로 popView 설정하였습니다.코스 부분 CVC만 스크롤이 됐었는데, CVC를 전체로 바꿔서 해결 했습니다.
📸 스크린샷
전에 남겼던 Comment
📮 관련 이슈