Skip to content

[SE-4473] Add progress bar views - #89

Merged
pkulkark merged 5 commits into
masterfrom
pooja/add-progress-bar-display
Nov 8, 2021
Merged

[SE-4473] Add progress bar views#89
pkulkark merged 5 commits into
masterfrom
pooja/add-progress-bar-display

Conversation

@pkulkark

Copy link
Copy Markdown
Member

Description:
This PR adds new views that display progress bars for course-level and chapter-level progress.
This requires the changes from open-craft/openedx-platform#431 to be present.

JIRA: SE-4473

Dependencies: open-craft/openedx-platform#431

Testing instructions:

  1. Once you have a edx devstack setup with changes from [SE-4473] Implement progress bar openedx-platform#431, install and enable completion tracking.
  2. Pip install this django app and checkout the PR branch.
  3. Make sure the feature flag "SHOW_PROGRESS_BAR" is set to True on your devstack.
  4. Start a new course in LMS and check that the two progress bars are displayed and updated correctly.

Reviewers:

  • [ ]

This adds new views to return progress bars
for course-level and chapter-level.
Comment thread completion_aggregator/urls.py
Comment thread completion_aggregator/views.py
Comment thread completion_aggregator/views.py
@pkulkark
pkulkark force-pushed the pooja/add-progress-bar-display branch from 1b82b04 to e141b96 Compare November 2, 2021 11:00

@gabor-boros gabor-boros left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍 🎉

Screenshot 2021-11-02 at 12 28 38

  • I tested what's in the test instructions
  • I read through the code

@pkulkark pkulkark changed the title WIP: [SE-4473] Add progress bar views [SE-4473] Add progress bar views Nov 3, 2021
@pkulkark
pkulkark force-pushed the pooja/add-progress-bar-display branch from e141b96 to ebe81ad Compare November 3, 2021 04:25
@pkulkark
pkulkark merged commit 2988ef9 into master Nov 8, 2021
@Agrendalath
Agrendalath deleted the pooja/add-progress-bar-display branch November 25, 2021 02:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants