openedx-unsupported / edx-app-android

The Open edX mobile app for Android!
https://docs.google.com/document/d/15MYZ2LAmahB82P5wkPkQOqjszfou0Qkeae3qz7STxUQ/edit
Apache License 2.0
301 stars 333 forks source link

refactor: Replace listView with recyclerView for Discussion Screen #1821

Closed omerhabib26 closed 1 year ago

omerhabib26 commented 1 year ago

Description

LEARNER-9578

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage has no change and project coverage change: -0.01% :warning:

Comparison is base (3c3eec3) 1.09% compared to head (ea928d9) 1.09%. Report is 4 commits behind head on app_nav.

:exclamation: Current head ea928d9 differs from pull request most recent head 8198682. Consider uploading reports for the commit 8198682 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## app_nav #1821 +/- ## ============================================ - Coverage 1.09% 1.09% -0.01% Complexity 137 137 ============================================ Files 536 536 Lines 25819 25862 +43 Branches 3300 3296 -4 ============================================ Hits 284 284 - Misses 25508 25551 +43 Partials 27 27 ``` | [Files Changed](https://app.codecov.io/gh/openedx/edx-app-android/pull/1821?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx) | Coverage Δ | | |---|---|---| | [...ain/java/org/edx/mobile/extenstion/ImageViewExt.kt](https://app.codecov.io/gh/openedx/edx-app-android/pull/1821?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx#diff-T3BlbkVkWE1vYmlsZS9zcmMvbWFpbi9qYXZhL29yZy9lZHgvbW9iaWxlL2V4dGVuc3Rpb24vSW1hZ2VWaWV3RXh0Lmt0) | `0.00% <0.00%> (ø)` | | | [...main/java/org/edx/mobile/extenstion/TextViewExt.kt](https://app.codecov.io/gh/openedx/edx-app-android/pull/1821?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx#diff-T3BlbkVkWE1vYmlsZS9zcmMvbWFpbi9qYXZhL29yZy9lZHgvbW9iaWxlL2V4dGVuc3Rpb24vVGV4dFZpZXdFeHQua3Q=) | `0.00% <0.00%> (ø)` | | | [...src/main/java/org/edx/mobile/extenstion/ViewExt.kt](https://app.codecov.io/gh/openedx/edx-app-android/pull/1821?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx#diff-T3BlbkVkWE1vYmlsZS9zcmMvbWFpbi9qYXZhL29yZy9lZHgvbW9iaWxlL2V4dGVuc3Rpb24vVmlld0V4dC5rdA==) | `0.00% <ø> (ø)` | | | [...rg/edx/mobile/model/discussion/DiscussionThread.kt](https://app.codecov.io/gh/openedx/edx-app-android/pull/1821?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx#diff-T3BlbkVkWE1vYmlsZS9zcmMvbWFpbi9qYXZhL29yZy9lZHgvbW9iaWxlL21vZGVsL2Rpc2N1c3Npb24vRGlzY3Vzc2lvblRocmVhZC5rdA==) | `0.00% <0.00%> (ø)` | | | [.../mobile/model/discussion/DiscussionTopicDepth.java](https://app.codecov.io/gh/openedx/edx-app-android/pull/1821?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx#diff-T3BlbkVkWE1vYmlsZS9zcmMvbWFpbi9qYXZhL29yZy9lZHgvbW9iaWxlL21vZGVsL2Rpc2N1c3Npb24vRGlzY3Vzc2lvblRvcGljRGVwdGguamF2YQ==) | `0.00% <0.00%> (ø)` | | | [...obile/src/main/java/org/edx/mobile/util/UiUtils.kt](https://app.codecov.io/gh/openedx/edx-app-android/pull/1821?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx#diff-T3BlbkVkWE1vYmlsZS9zcmMvbWFpbi9qYXZhL29yZy9lZHgvbW9iaWxlL3V0aWwvVWlVdGlscy5rdA==) | `0.00% <ø> (ø)` | | | [...mobile/view/CourseDiscussionPostsBaseFragment.java](https://app.codecov.io/gh/openedx/edx-app-android/pull/1821?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx#diff-T3BlbkVkWE1vYmlsZS9zcmMvbWFpbi9qYXZhL29yZy9lZHgvbW9iaWxlL3ZpZXcvQ291cnNlRGlzY3Vzc2lvblBvc3RzQmFzZUZyYWdtZW50LmphdmE=) | `0.00% <0.00%> (ø)` | | | [...bile/view/CourseDiscussionPostsSearchFragment.java](https://app.codecov.io/gh/openedx/edx-app-android/pull/1821?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx#diff-T3BlbkVkWE1vYmlsZS9zcmMvbWFpbi9qYXZhL29yZy9lZHgvbW9iaWxlL3ZpZXcvQ291cnNlRGlzY3Vzc2lvblBvc3RzU2VhcmNoRnJhZ21lbnQuamF2YQ==) | `0.00% <0.00%> (ø)` | | | [...bile/view/CourseDiscussionPostsThreadFragment.java](https://app.codecov.io/gh/openedx/edx-app-android/pull/1821?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx#diff-T3BlbkVkWE1vYmlsZS9zcmMvbWFpbi9qYXZhL29yZy9lZHgvbW9iaWxlL3ZpZXcvQ291cnNlRGlzY3Vzc2lvblBvc3RzVGhyZWFkRnJhZ21lbnQuamF2YQ==) | `0.00% <0.00%> (ø)` | | | [...mobile/view/CourseDiscussionResponsesFragment.java](https://app.codecov.io/gh/openedx/edx-app-android/pull/1821?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx#diff-T3BlbkVkWE1vYmlsZS9zcmMvbWFpbi9qYXZhL29yZy9lZHgvbW9iaWxlL3ZpZXcvQ291cnNlRGlzY3Vzc2lvblJlc3BvbnNlc0ZyYWdtZW50LmphdmE=) | `0.00% <ø> (ø)` | | | ... and [8 more](https://app.codecov.io/gh/openedx/edx-app-android/pull/1821?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx) | | ... and [5 files with indirect coverage changes](https://app.codecov.io/gh/openedx/edx-app-android/pull/1821/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.