frontend-app-learning icon indicating copy to clipboard operation
frontend-app-learning copied to clipboard

Sidebar weird behavior when course is not completly live/scheduled

Open mariajgrimaldi opened this issue 1 year ago • 5 comments

Expected behavior

As a user, I want to be able to navigate between sections if I'm on the locked unit so that I can easily return to the section with the prerequisite and finish the assignment

Actual behavior

As a user, I CANNOT navigate between sections if I'm on the locked unit so that I have to return to the outline and spend additional time trying to figure out what is going on

Steps to reproduce

  1. Enrol into the course with subsections with prerequisites (I created this one for test).
  2. Start with any subsection.
  3. Go to the "locked unit" - with the pre-requisite on the subsection.
  4. There are no sections and subsections, just a "course outline".

https://github.com/openedx/wg-build-test-release/assets/91068576/bf0a97f4-d90d-4e29-8d33-198147cb24b9

Additional information

I'm testing it with the staff user, through Audit mode (maybe it affects smth).

As a student, this happened: https://github.com/openedx/wg-build-test-release/issues/363#issuecomment-2129799931. I don't think this is related to the sidebar itself.

This issue was reported as part of the redwood testing process: https://github.com/openedx/frontend-app-learning/issues/1385

mariajgrimaldi avatar Jun 02 '24 15:06 mariajgrimaldi

@crathbun428 helped us test even further and found:

I tested this as a student and got the same error message. I went into the course in studio as staff and published and scheduled everything that had not been published or scheduled for the course (most of the locked and prerequisite units). Once everything in the course was published and scheduled, this weird behavior did not appear for either the learner account or the staff/admin account. This behavior only appears when there are locked or prerequisite units that haven't been scheduled in the course.

So I'm opening this issue in case this shouldn't be happening.

mariajgrimaldi avatar Jun 02 '24 15:06 mariajgrimaldi

FYI @arbrandes @brian-smith-tcril @openedx/2u-infinity, please tag anyone you think can help. Thanks!

mariajgrimaldi avatar Jun 03 '24 16:06 mariajgrimaldi

@GlugovGrGlib: do you have any insights on this issue? We'd much appreciate it!

mariajgrimaldi avatar Jun 06 '24 02:06 mariajgrimaldi

Sorry, missed this one, will look into it latter next week

GlugovGrGlib avatar Jun 19 '24 18:06 GlugovGrGlib

Hey @GlugovGrGlib, let me know if you have any updates on this issue. I'll assign it to you for the time being. If you can't work on it, you can unassign yourself. Thanks!

mariajgrimaldi avatar Aug 23 '24 18:08 mariajgrimaldi

@mariajgrimaldi @GlugovGrGlib I think I may have a fix for this.

Image

wgu-jesse-stewart avatar Apr 15 '25 15:04 wgu-jesse-stewart

I can see that it is displaying after publishing. But it also will display after changing and saving the Section or Module name.

Steps to reproduce. 1 - change unit name and save (do not publish). 2 - refresh course and notice unit name is not changed. 3 - change section or module name (do not publish). 4 - refresh course and notice the until name DID change. 5 - go back to course authoring tool and notice the unit changes have not been published.

This is only true for the Unit title. the Unit content does not update until the unit changes are "published". The title always updates when section and module are saved, no publishing required.

https://github.com/user-attachments/assets/ecd78437-f444-49b9-bfbc-8af9609daad9

wgu-jesse-stewart avatar Apr 30 '25 16:04 wgu-jesse-stewart

@wgu-jesse-stewart: thank you for taking this on! Is this comment also related to this issue or to a different one?

Also, does this PR #1679 fixes the navigation between locked sections?

mariajgrimaldi avatar May 15 '25 08:05 mariajgrimaldi

@mariajgrimaldi I do not think my PR fixes this issue. However, it does fix the locked sections navigation issue.

wgu-jesse-stewart avatar May 15 '25 16:05 wgu-jesse-stewart