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

[AN] fix: 각종 에러 처리 + 리팩터링 #359

Merged
merged 12 commits into from
Aug 12, 2024
Merged

Conversation

gaeun5744
Copy link
Member

@gaeun5744 gaeun5744 commented Aug 11, 2024

이슈

개발 사항

  • mainActivity: nav 선택 위치와 띄워진 fragment가 동일하도록 구현
  • ShapeableImageView 전체 적용
  • refresh 에러 해결
  • 강아지 태어난 달 bottomSheet 클릭 안되는 이슈 해결

전달 사항 (없으면 삭제해 주세요)

  • 이슈 #353의 경우, 앱이 백그라운드에 오래 있으면 저절로 메모리에서 지워지네요..? 권한 쪽 문제는 아닙니다ㅠ

@gaeun5744 gaeun5744 added 🐞 bug 버그 수정 🛠️ refactor 리팩터링 🤖 android android labels Aug 11, 2024
@gaeun5744 gaeun5744 self-assigned this Aug 11, 2024
Copy link

github-actions bot commented Aug 11, 2024

Test Results

3 tests  ±0   3 ✅ ±0   0s ⏱️ ±0s
1 suites ±0   0 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit 6ddd350. ± Comparison against base commit 285323a.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@jinuemong jinuemong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다 !!

@@ -41,7 +41,7 @@ class MainActivity :
}

private fun initNavController() {
switchFragment(ClubListFragment::class.java)
supportFragmentManager.findFragmentById(R.id.nav_host_fragment_container) ?: switchFragment(ClubListFragment::class.java)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[mainActivity: nav 선택 위치와 띄워진 fragment가 동일하도록 구현]에 대한 fix인가요?
이유가 무엇이었을까요

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

현재 container에 fragment가 있는지 없는지 확인하는 코드입니다! 해당 코드로 configuration change때 무분별하게 fragment가 switch되는 것을 막을 수 있습니다

@@ -12,7 +12,7 @@ fun ChatRoom.toUiModel(): ChatListUiModel =
"",
memberCount,
0,
LocalDateTime.now().classifyChatDateTime(),
LocalDateTime.of(2024, 6, 7, 13, 6).classifyChatDateTime(),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인부탁드려요~~

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

아 일부러 더미데이터로 넣었습니다!! listAdapter 데이터 갱신 기준 확인할려구요

@gaeun5744 gaeun5744 merged commit 44bff4b into develop Aug 12, 2024
2 checks passed
@gaeun5744 gaeun5744 deleted the hotfix/fix-error branch August 20, 2024 02:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖 android android 🐞 bug 버그 수정 🛠️ refactor 리팩터링
Projects
None yet
3 participants