bugfix - Course discovery facet

Description

In case of custom facet, specially for boolean fields the selected facet not highlighted.

See the following code snippet, the Online facet not highlighted:

'enable_conferance': {
'name': _('Course Status'),
'terms': {
'true': _('Online'),
'false': _('Offline'),
},
},

Steps to Reproduce

None

Activity

Show:
ChristinaR
June 6, 2016, 4:06 PM

It is my understanding that course discovery is owned by the Solutions team. FYI and .

BTW, as this is a bug fix, I would expect a test that fails without the fix and passes with the fix. But I don't have much context here about the feasibility of a test in this particular case.

Dadaso Zanzane
June 7, 2016, 8:28 AM

This error occurred because facet data is Boolean field.

I my case, I want to search the course with fields as follows -
Online Course = True
Offline Course = False

So I was able to search the data, but the label get not highlighted. see the example below.

So I have resolved issue -
https://github.com/edx/edx-platform/pull/12545/files

Please review,
Thanks and Regards,
-Dadaso

Steve Magoun
June 7, 2016, 1:39 PM

There is no active development on course discovery search right now, and the original author of this section of code is no longer working on Open edX. I suggest that and review this PR.

Marko Jevtic
June 9, 2016, 3:11 PM

I can take a look at this PR as one of the reviewers.

fatma
December 3, 2016, 11:36 AM

please help
how can i add new course discovery facet at openedx search ?

Assignee

Unassigned

Reporter

Dadaso Zanzane

Platform Area

None

Priority

Unset
Configure