wg-build-test-release icon indicating copy to clipboard operation
wg-build-test-release copied to clipboard

[Quince Testing] <TC_AUTHOR_133>: <Course still showing in catalog page even "none" property set in >

Open Faisalarbisoft opened this issue 2 years ago • 6 comments

Release

Quince

Expected behavior

Studio Advance settings, "Course Visibility In Catalog" defines when "none" value is set then course should not show in catalog and not allows the access to an about page.

Actual behavior

Course is still displaying in the catalog even "none" value is set against "Course Visibility In Catalog"

Steps to reproduce

  1. Go to studio select any course and go to its Advanced Settings, set the value of "Course Visibility In Catalog" to "none" and save it
  2. Go to Lms and check the course catalog page.

Additional information

Screenshot 2023-11-23 at 4 03 26 PM Screenshot 2023-11-23 at 4 07 20 PM

Faisalarbisoft avatar Nov 23 '23 11:11 Faisalarbisoft

label: quince testing

Faisalarbisoft avatar Nov 23 '23 11:11 Faisalarbisoft

@Faisalarbisoft: I found a similar issue reported a while ago that's worth reading: https://github.com/openedx/wg-build-test-release/issues/164

I'll tag @ghassanmas and @regisb since they were around by the time of the first report. Could you give us a hand here? Thanks!

mariajgrimaldi avatar Dec 07 '23 05:12 mariajgrimaldi

To the best of my recollection, this is not a bug but a feature that is implemented in a very confusing way. Setting the catalogue visibility as "none" is not meant to affect how the courses are displayed in the /courses page. But I think that we should change that behaviour, because it's really not what the user would expect.

regisb avatar Feb 08 '24 07:02 regisb

Hello @fayyazahmed66! You marked this test as passing in the Redwood spreadsheet. You could add the steps you followed here and in the testing spreadsheet to move this forward. Thanks!

mariajgrimaldi avatar May 24 '24 19:05 mariajgrimaldi

@mariajgrimaldi thanks for pointing out. I have missed this use case and test with "both" only. Let me retest.

fayyazahmed66 avatar May 25 '24 13:05 fayyazahmed66

@jmakowski1123: A few months ago, we discussed this issue in the BTR WG, and the consensus was to change the strategy we've been using until now and involve the product WG in defining what this feature actually is. Can you confirm that the product WG will take a look at it? Let me know whether I remember it correctly. Thanks!

mariajgrimaldi avatar Aug 06 '24 14:08 mariajgrimaldi