feat: [FC-0092] Course progress tab #448

PavloNetrebchuk · 2025-07-09T10:09:12Z

Added a progress screen to display user’s course and assignment completion status
Support RTL and A11Y

In that PR, color coding is mocked. It will work correctly once the backend returns the color list.
val defaultColors = listOf("#fe553a", "#32c0ff", "#a3ff7b", "#ff30ee")

Light	Dark

openedx-webhooks · 2025-07-09T10:09:18Z

Thanks for the pull request, @PavloNetrebchuk!

This repository is currently maintained by @openedx/openedx-mobile-maintainers.

Once you've gone through the following steps feel free to tag them in a comment and let them know that your changes are ready for engineering review.

🔘 Get product approval

If you haven't already, check this list to see if your contribution needs to go through the product review process.

If it does, you'll need to submit a product proposal for your contribution, and have it reviewed by the Product Working Group.
- This process (including the steps you'll need to take) is documented here.
If it doesn't, simply proceed with the next step.

🔘 Provide context

To help your reviewers and other members of the community understand the purpose and larger context of your changes, feel free to add as much of the following information to the PR description as you can:

Dependencies

This PR must be merged before / after / at the same time as ...
Blockers

This PR is waiting for OEP-1234 to be accepted.
Timeline information

This PR must be merged by XX date because ...
Partner information

This is for a course on edx.org.
Supporting documentation
Relevant Open edX discussion forum threads

🔘 Get a green build

If one or more checks are failing, continue working on your changes until this is no longer the case and your build turns green.

Details

Where can I find more information?

If you'd like to get more details on all aspects of the review process for open source pull requests (OSPRs), check out the following resources:

When can I expect my changes to be merged?

Our goal is to get community contributions seen and reviewed as efficiently as possible.

However, the amount of time that it takes to review and merge a PR can vary significantly based on factors such as:

The size and impact of the changes that it introduces
The need for product review
Maintenance status of the parent repository

💡 As a result it may take up to several weeks or months to complete a review and merge your PR.

core/src/main/java/org/openedx/core/data/api/CourseApi.kt

core/src/main/java/org/openedx/core/data/model/CourseProgressResponse.kt

core/src/main/java/org/openedx/core/data/model/room/CourseProgressEntity.kt

course/src/main/java/org/openedx/course/presentation/progress/CourseProgressScreen.kt

volodymyr-chekyrta

Please address Detekt issues as well.

core/src/main/java/org/openedx/core/data/storage/CourseDao.kt

core/src/main/java/org/openedx/core/domain/model/CourseProgress.kt

course/src/main/res/drawable/course_ic_marker.xml

course/src/main/res/values/strings.xml

core/src/main/java/org/openedx/core/data/storage/CourseDao.kt

volodymyr-chekyrta

LGTM 👍 , we can proceed with testing and external review of this PR.

…gress_tab # Conflicts: # course/src/main/java/org/openedx/course/presentation/progress/CourseProgressScreen.kt

edschema · 2025-07-25T14:02:55Z

Android PR associated with Issue: Course Progress Page #625

edschema · 2025-08-04T13:39:10Z

Screenshots look good. Passed to community code review July 31

RawanMatar89

LGTM

RawanMatar89 · 2025-08-15T15:44:12Z

core/src/main/java/org/openedx/core/data/model/CourseProgressResponse.kt

+        @SerializedName("grade_range") val gradeRange: Map<String, Float>?,
+        @SerializedName("assignment_colors") val assignmentColors: List<String>?
+    ) {
+        // Temporary solution. Backend will returns color list later


can we add TODO here

core/src/main/java/org/openedx/core/data/model/CourseProgressResponse.kt

PavloNetrebchuk added 5 commits June 19, 2025 16:18

feat: progress tab UI

48d6398

feat: progress tab logic

ec59fb4

feat: cache first logic

6e23b39

feat: color coding

38977a2

fix: changes according demo feedback

284561e

openedx-webhooks added the open-source-contribution PR author is not from Axim or 2U label Jul 9, 2025

openedx-webhooks added this to Contributions Jul 9, 2025

github-project-automation bot moved this to Needs Triage in Contributions Jul 9, 2025

PavloNetrebchuk requested a review from IvanStepanok July 9, 2025 10:09

PavloNetrebchuk self-assigned this Jul 9, 2025

github-advanced-security bot found potential problems Jul 9, 2025

View reviewed changes

PavloNetrebchuk marked this pull request as draft July 9, 2025 10:11

fix: changes according demo feedback

034d8d5

volodymyr-chekyrta requested changes Jul 10, 2025

View reviewed changes

PavloNetrebchuk force-pushed the feat/progress_tab branch 4 times, most recently from 51779d8 to b535314 Compare July 11, 2025 09:42

github-advanced-security bot found potential problems Jul 11, 2025

View reviewed changes

core/src/main/java/org/openedx/core/data/storage/CourseDao.kt Fixed Show fixed Hide fixed

PavloNetrebchuk force-pushed the feat/progress_tab branch from b535314 to 2ffa9df Compare July 11, 2025 10:09

volodymyr-chekyrta previously approved these changes Jul 11, 2025

View reviewed changes

PavloNetrebchuk dismissed volodymyr-chekyrta’s stale review via c21c1bd July 11, 2025 14:44

PavloNetrebchuk force-pushed the feat/progress_tab branch from 2ffa9df to c21c1bd Compare July 11, 2025 14:44

Merge remote-tracking branch 'origin/feat/progress_tab' into feat/pro…

da8c768

…gress_tab # Conflicts: # course/src/main/java/org/openedx/course/presentation/progress/CourseProgressScreen.kt

PavloNetrebchuk force-pushed the feat/progress_tab branch from c21c1bd to da8c768 Compare July 11, 2025 15:00

mphilbrick211 added the FC Relates to an Axim Funded Contribution project label Jul 15, 2025

mphilbrick211 moved this from Needs Triage to Waiting on Author in Contributions Jul 15, 2025

PavloNetrebchuk force-pushed the feat/progress_tab branch 2 times, most recently from 2cdcc80 to 8e4bc7b Compare July 18, 2025 10:22

PavloNetrebchuk marked this pull request as ready for review July 18, 2025 10:35

PavloNetrebchuk force-pushed the feat/progress_tab branch 2 times, most recently from 7a50a03 to 8703fd2 Compare July 21, 2025 17:11

volodymyr-chekyrta mentioned this pull request Jul 28, 2025

feat: [FC-0092] Course content tabs #450

Merged

feat: color coding

924461f

PavloNetrebchuk force-pushed the feat/progress_tab branch from 8703fd2 to 924461f Compare July 28, 2025 19:20

feat: separator between assignment policies on the progress

381b8b3

PavloNetrebchuk force-pushed the feat/progress_tab branch from 4b02e71 to 381b8b3 Compare July 31, 2025 10:29

feat: progressTabClickedEvent

b970fb5

edschema mentioned this pull request Aug 14, 2025

[GH Request] Onboard @RawanMatar89 openedx/axim-engineering#1516

Closed

edschema requested a review from RawanMatar89 August 15, 2025 12:34

edschema mentioned this pull request Aug 15, 2025

Course Progress Page openedx/openedx-app-ios#625

Closed

2 tasks

RawanMatar89 approved these changes Aug 15, 2025

View reviewed changes

PavloNetrebchuk force-pushed the feat/progress_tab branch from 8c9d166 to eafcd7b Compare August 18, 2025 10:03

github-advanced-security bot found potential problems Aug 18, 2025

View reviewed changes

core/src/main/java/org/openedx/core/data/model/CourseProgressResponse.kt Fixed Show fixed Hide fixed

feat: defaultColors changed

331c5e0

PavloNetrebchuk force-pushed the feat/progress_tab branch from eafcd7b to 331c5e0 Compare August 18, 2025 10:05

volodymyr-chekyrta approved these changes Aug 18, 2025

View reviewed changes

volodymyr-chekyrta merged commit 4df0b02 into openedx:develop Aug 18, 2025
6 checks passed

github-project-automation bot moved this from Waiting on Author to Done in Contributions Aug 18, 2025

volodymyr-chekyrta deleted the feat/progress_tab branch August 18, 2025 10:14

PavloNetrebchuk mentioned this pull request Oct 23, 2025

Release v2.2 (Ulmo.1) #465

Merged

feat: [FC-0092] Course progress tab #448

feat: [FC-0092] Course progress tab #448

Uh oh!

Conversation

PavloNetrebchuk commented Jul 9, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

openedx-webhooks commented Jul 9, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

volodymyr-chekyrta left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

volodymyr-chekyrta left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

edschema commented Jul 25, 2025

Uh oh!

edschema commented Aug 4, 2025

Uh oh!

RawanMatar89 left a comment

Choose a reason for hiding this comment

Uh oh!

RawanMatar89 Aug 15, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

PavloNetrebchuk commented Jul 9, 2025 •

edited

Loading

openedx-webhooks commented Jul 9, 2025 •

edited

Loading

volodymyr-chekyrta left a comment •

edited

Loading