Skip to content
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

Refactor/#861 network view model 적용 #873

Merged
merged 100 commits into from
Dec 12, 2023

Conversation

ki960213
Copy link
Collaborator

@ki960213 ki960213 commented Dec 12, 2023

#️⃣ 연관된 이슈

close : #861

📝 작업 내용

  • MemberBlockViewModel
  • ChildCommentViewModel
  • CompetitionFilterViewModel
  • CompetitionViewModel
  • ConferenceFilterViewModel
  • ConferenceViewModel
  • EditMyProfileViewModel
  • EventDetailViewModel
  • EventSearchViewModel
  • FeedDetailViewModel
  • FeedListViewModel
  • FeedWritingViewModel
  • LoginViewModel
  • MessageListViewModel
  • MessageRoomViewModel
  • MyCommentsViewModel
  • MyProfileViewModel
  • MyRecruitmentViewModel
  • NotificationConfigViewModel
  • NotificationTagConfigViewModel
  • OnboardingViewModel
  • PrimaryNotificationViewModel
  • ProfileViewModel
  • RecruitmentPostDetailViewModel
  • EventRecruitmentViewModel
  • RecruitmentPostWritingViewModel
  • ScrappedEventViewModel
  • SettingViewModel
  • SplashViewModel

변경된 기능

공통:

  • 모든 command 요청 시 서버로부터 응답이 오기 전까지 command 요청을 다시 할 수 없도록 변경
  • 모든 새로고침 시 Progressbar 안보이게 변경
  • SwipeRefreshLayout 사용할 경우 새로고침 시 해당 뷰의 애니메이션과 동기화 되도록 변경
  • 모든 화면에서 데이터 불러오기 실패하면 실패 알림 다이얼로그 띄우고 확인 시 화면 나가게 하는 기능 구현

FeedDetail:

  • 댓글 수정 시 그 댓글이 가장 위로 스크롤 되고 배경색이 변함. 수정할 동안 계속 유지됨. 이것을 하이라이팅이라는 기능으로 명명

FeedWriting:

  • 글을 조금이라도 작성했다면 화면을 나가기 전에 ConfirmDialog를 띄움. 변하지 않았다면 바로 나감.
  • 제목과 content가 비어있지 않다면 등록 버튼이 활성화됨. 그렇지 않으면 비활성화 됨.

MessageList:

  • MessageListActivity가 화면에 보여지고 있을 때 다른 MessageRoom의 메세지가 오면 알림이 띄워지게 만듦.
  • MessageListActivity가 화면에 보여지고 있을 때 해당 MessageRoom의 메세지가 왔을 때 만약 아래로 스크롤 할 수 없다면 자동으로 새로운 메세지가 보이게 스크롤 됨. 그렇지 않으면 새로운 메세지 ui를 보여줌.

MyCommentList:

  • 내 댓글 클릭하면 ChildCommentActivity 대신 FeedDetailActivity를 실행하도록 변경

NotificationConfig:

  • 같이가요 알림 지움

NotificationList:

  • 횡스크롤 못하게 ViewPage 지움
  • 댓글 알림 클릭하면 ChildCommentActivity 대신 FeedDetailActivity를 실행하도록 변경

NotificationTag:

  • 태그 설정이 변했을 때만 등록하기 버튼이 활성화되도록 변경
  • 태그 설정이 변했을 때만 나갈 때 ConfirmDialog를 띄우도록 변경

Onboarding:

  • 구성 변경에 대응하기 위해 스크롤 되도록 변경

RecruitmentWriting:

  • 글이 작성되었을 때만 등록 버튼이 활성화 되도록 변경
  • 글이 작성되었을 때 나가려고 하면 ConfirmDialog 띄우도록 변경
  • 수정할 때 글이 수정되었을 때만 나갈 때 ConfirmDialog 띄우는 기능 추가

Splash:

  • 스플래시 애니메이션 시간 동안만 보여지도록 변경

KerdyFirebaseMessagingService:

  • 댓글 알림 오면 ChildCommentActivity 대신 FeedDetailActivity를 실행하도록 변경

눈에 보이는 큰 변경사항은 위와 같고 세부적인 UI나 코드 형식, 세부적인 로직도 많이 바뀌었습니다. 확인해주세요.

스크린샷 (선택)

예상 소요 시간 및 실제 소요 시간 (일 / 시간 / 분)

예상 소요 시간 : 3일
실제 소요 시간 : 2주

💬 리뷰어 요구사항 (선택)

게시글 작성 화면에서 화면 높이가 작으면 사진에 의해 content EditText가 많이 가려집니다. 이것만 해결해주세요. 😢

ki960213 and others added 30 commits December 1, 2023 18:58
…용' into Refactor/#861-NetworkViewModel_적용

# Conflicts:
#	android/2023-emmsale/app/src/main/java/com/emmsale/presentation/ui/scrappedEventList/ScrappedEventViewModel.kt
@ki960213 ki960213 added Android 안드로이드 관련 이슈 리팩터링 테스트 코드의 검증 값이 변환하지 않고 코드 변경 기능 추가 새로운 기능 추가 및 기존 기능 변경 High Priority 리뷰 우선순위가 높은 PR labels Dec 12, 2023
@ki960213 ki960213 self-assigned this Dec 12, 2023
@chws0508 chws0508 merged commit fa38c87 into android-main Dec 12, 2023
1 check passed
@ki960213 ki960213 deleted the Refactor/#861-NetworkViewModel_적용 branch December 12, 2023 11:24
@ki960213 ki960213 restored the Refactor/#861-NetworkViewModel_적용 branch December 13, 2023 06:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Android 안드로이드 관련 이슈 High Priority 리뷰 우선순위가 높은 PR 기능 추가 새로운 기능 추가 및 기존 기능 변경 리팩터링 테스트 코드의 검증 값이 변환하지 않고 코드 변경
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants