fix: bug when sequence units level appear after tapping back #260
Conversation
There was a problem hiding this comment.
We need to make changes while considering the COURSE_NESTED_LIST_ENABLED flag, accounting for both true and false values.
Previously, it functioned correctly with the false value, but now it operates with the true value. Let's account for both scenarios this time. 😄
|
@HamzaIsrar12 so, when COURSE_NESTED_LIST_ENABLED = false when an user clicks the resume button the next content will be opened and then when he goes back the list of units should appear? Don't think this is a good behaviour...thoughts? |
|
@dixidroid |
dixidroid
force-pushed
the
fix/back_from_resume_and_dates
branch
from
96269ed to
76b6615
Compare
|
@HamzaIsrar12 I've added an ability to use the COURSE_NESTED_LIST_ENABLED flag |
dixidroid
force-pushed
the
fix/back_from_resume_and_dates
branch
from
76b6615 to
90344a2
Compare
| @@ -168,14 +168,24 @@ class CourseOutlineFragment : Fragment() { | |||
| viewModel.resumeSectionBlock?.let { subSection -> | |||
There was a problem hiding this comment.
We can also move this and the other block (in DatesFragment) to their respective ViewModel, but that's up to you now. 😄
Removed sequence units level when open a content from Resume or Dates