버그 리포트

[Yoon6]

버그 발견되면 적어주세요.

[Yoon6]

정책상 문제

• 운동을 시작한 상태에서 루틴을 수정했을 때, 이미 히스토리를 만들었기 때문에 반영 안됨

[Yoon6]

기존에 설정된 루틴의 운동 내용을 변경하면, 홈화면에서 바로 반영 안됨. 나갔다 들어오면 반영됨

[Yoon6]

루틴 가져오기 할 떄, exercise,set 순서 유지 안됨

@sky8492002

[Yoon6]

• 가져오기한 루틴 선택 후 삭제 시 터짐

• 3분할 평범 루틴 가져와서 삭제할 때 터지네요

• 가져온 루틴 1개 제외하고는 다른 루틴은 되네요...

  • 똑같은거 가져와도 잘 지워지고, 가져와있는 1개만 안되네요. 인스펙터 만진적도 없는데.

@sky8492002

[Yoon6]

• 운동 완료 후, 선택된 루틴이 없으면 홈화면에 complete만 찍힘

문제는 아니지만, 어떻게 표시해야할 지 결정해야할 듯

[Yoon6]

[Yoon6]

로컬 루틴 상세화면에서 리사이클러뷰 expand안됨

[jaeseuk]

기존에 설정된 루틴의 운동 내용을 변경하면, 홈화면에서 바로 반영 안됨. 나갔다 들어오면 반영됨

이거 수정했다고 하지 않으셨던가요?

[jaeseuk]

로컬 루틴 상세화면에서 리사이클러뷰 expand안됨

이거 코드 한줄 없어져서 그렇더라구요.
근데 그 코드가 의미적으로 맞지 않는 거 같아서 지웠는데 일단 동작하게 하려면 다시 추가하거나 구조를 변경하거나 해야 할 것 같습니다.

[TRASALBY]

루틴 가져오기 할 떄, exercise,set 순서 유지 안됨

@sky8492002

이 부분도 저번에 제가 말씀 드린거 같은데 해결했다고 하시지 않았나요?

[TRASALBY]

운동이름에도 제한이 있어야겠네요

[Yoon6]

기존에 설정된 루틴의 운동 내용을 변경하면, 홈화면에서 바로 반영 안됨. 나갔다 들어오면 반영됨

이거 수정했다고 하지 않으셨던가요?

다시 테스트 해보니까 되네요.
갑자기 안되길래 적어놨는데, 제 착각이었는지, 다시 발견하면 추가하겠습니다.

[jaeseuk]

운동이름에도 제한이 있어야겠네요

이 부분은 운동이름 제한도 그렇지만 저 레이아웃 크기나 제약에도 문제가 있어 보이네요.

[jaeseuk]

기존에 설정된 루틴의 운동 내용을 변경하면, 홈화면에서 바로 반영 안됨. 나갔다 들어오면 반영됨

이거 수정했다고 하지 않으셨던가요?

다시 테스트 해보니까 되네요. 갑자기 안되길래 적어놨는데, 제 착각이었는지, 다시 발견하면 추가하겠습니다.

수정 한 부분이 sprint4에 있나요? 지금 코드로는 안되는게 맞는 거 같은데요?

[Yoon6]

기존에 설정된 루틴의 운동 내용을 변경하면, 홈화면에서 바로 반영 안됨. 나갔다 들어오면 반영됨

이거 수정했다고 하지 않으셨던가요?

다시 테스트 해보니까 되네요. 갑자기 안되길래 적어놨는데, 제 착각이었는지, 다시 발견하면 추가하겠습니다.

수정 한 부분이 sprint4에 있나요? 지금 코드로는 안되는게 맞는 거 같은데요?

네 sprint4 내용인데, 저는 지금은 잘 동작하는데 혹시 안되시나요

[sky8492002]

루틴 가져오기 할 떄, exercise,set 순서 유지 안됨
@sky8492002

이 부분도 저번에 제가 말씀 드린거 같은데 해결했다고 하시지 않았나요?

확인해보니 열람 화면에서는 괜찮은데 생성 / 수정 화면에서 순서가 바뀌네요.. 생성/수정 화면에서 exercise를 표시할 때 order 순으로 정렬이 되지 않아서일까요?

코드를 확인해보니 열람 화면과 관련된 DetailExerciseAdapter는

세트를 정렬 후 submitlist 하는 반면
생성 / 수정 화면과 관련된 RoutineExerciseAdapter는

정렬하지 않고 submitlist하고 있어서 발생하는 문제였네요 저기에 .sortedBy { it.order }만 추가하면 해결될 것 같습니다!

[jaeseuk]

네 sprint4 내용인데, 저는 지금은 잘 동작하는데 혹시 안되시나요

네 안돼요.
그리고 코드를 보면 반환타입이 flow나 livedata이 아니라서 변경 감지를 못 하기 때문에 반영 안되는게 맞는 것 같습니다.

[Yoon6]

HomeFragment.kt

    private fun setAdapter() {
        collectOnLifecycle {
            viewModel.selectedDay.collect { dayUiModel ->
                val exercises = dayUiModel?.exercises ?: emptyList()
                val homeAdapters = exercises.map { exerciseUiModel ->
                    HomeAdapter(exerciseUiModel)
                }

                adapter = ConcatAdapter(homeAdapters)
                binding.layoutDayExercises.recyclerViewTodayRoutine.adapter = adapter
                binding.layoutDayExercises.recyclerViewTodayRoutine.itemAnimator = ExpandableItemAnimator()
            }
        }
    }

HomeViewModel.kt

    val selectedDay = userInfo.map {
        val dayId = it?.dayId
        if (dayId.isNullOrEmpty()) return@map null
        val dayWithExercises = routineRepository.getDayWithExercisesByDayId(it.dayId)
        dayWithExercises.day.toDayUiModel(
            dayWithExercises.day.order,
            dayWithExercises.exercises
        )
    }.stateIn(viewModelScope, SharingStarted.Lazily, null)

반환타입 StateFlow<DayUiModel?>로 되어있는데, 홈화면 보시는 거 맞나요?

[jaeseuk]

반환타입 StateFlow<DayUiModel?>로 되어있는데, 홈화면 보시는 거 맞나요?

네 맞습니다. 혹시 선택한 루틴 정보 변경해보셨나요?
저 dayWithExercises가 flow가 아니라 변경 사항 반영 안될텐데요.

[jaeseuk]

루틴 관리 화면

• 루틴 개수 제한이 없는데 페이징 처리도 안하고 있어 루틴이 많이 추가되면 문제가 생길 것 같다.

[TRASALBY]

루틴 관리 화면

• 루틴 개수 제한이 없는데 페이징 처리도 안하고 있어 루틴이 많이 추가되면 문제가 생길 것 같다.

보통 운동을 한다고 생각할 때 몇개 정도의 루틴을 가지고 있을까요? 사실 제 생각은 그렇게 많이 가지고 있을 필요는 없다고 생각해서 많아도 20개까지면 충분하지 않을까 싶긴한데 그러면 제한을 거는것도 좋아 보입니다.

[TRASALBY]

공유된 루틴 가져오기 기능이 수행되면 해당 루틴에 대한 정보만 서버에 요청해서 룸에 저장되있는 공유 횟수를 바꾸면 어떨까요?
@sky8492002

[sky8492002]

공유된 루틴 가져오기 기능이 수행되면 해당 루틴에 대한 정보만 서버에 요청해서 룸에 저장되있는 공유 횟수를 바꾸면 어떨까요? @sky8492002

오 그러면 커뮤니티 화면으로 돌아올 때 실시간 공유 횟수를 확인할 수 있겠네요 한번 시도해 보겠습니다!