We were a little too ham-handed with the change in the linked ticket, and need to ensure that we only show the archive run if it is the ONLY run present.
We will likely need to revert most of the JS changes done on this PR:
https://github.com/edx/edx-mktg/pull/1406/files#diff-4bc02f80c98327e78f1b7e4e2cc57698
Example:
https://www.edx.org/micromasters/columbiax-artificial-intelligence
Please take note of this work by Mike. I think this is a requirement miss from
so what i understand from this ticket if there are multiple course runs we should not show any course run if all are expired. or this code is showing all course_runs which are expired. can you share a example or error.
Sure - I think this is what the 2nd acceptance criteria on was trying to cover. If there are ANY enrollable/upgradable/verifiable course runs, we should not use the archived course in the horizontal course run display.
This program shows the issue:
https://www.edx.org/micromasters/columbiax-artificial-intelligence
The top 2 course runs are still upgradeable and the bottom two course runs are archived. We should be showing the next course run for the bottom two courses.
I'll add a screenshot in case i fix this before EOD.
i just had a look at the pr and can not understand how this is related to LEARNER-695.
it had to do with the course run level filtering removed in the discovery_card_horizontal_view, but I have put that back