Auto-suggest course content on search (Meilisearch-compatible)

Development
21

@braden Thank you for informing me. I have been working on creating a proof of concept (POC) for courseware search. You can view the changes I have made to integrate Meilisearch into the learning MFE here.

To support the above implementation, I have also created an additional abstraction layer, which you can find here. It is not yet ready for review as I need to rename the package and perform some refactoring. However, I am sharing it to provide insight into the direction I am taking. Please let me know if you find it interesting or if you have any suggestions or additions.

here I have also added a content class in openedx to feed data to indexer.

image
image2796×1558 351 KB

22

@braden I have drafted an Architectural Decision Record (ADR) for our project, which you can review here. The document includes context from your previous ADR, and I have acknowledged your contributions accordingly.

Please let me know if there are any adjustments or additions you would recommend.

Thank you for your guidance and support.

23

@qasimgulzar Do you mean Meilisearch ADR by qasimgulzar · Pull Request #1 · qasimgulzar/openedx-search-api · GitHub ? That one seems identical to my previous ADR and doesn’t really explain the rationale for a new abstraction layer nor the architecture that it should have. It would be nice if you could write that up and get buy-in from folks.

24

@Braden, I apologize for the oversight. The document was missing essential information from the ADR. Thank you for bringing this to my attention. I have updated the document accordingly.

Could you please review the updated version when you have some free time? If you notice anything else that might be missing, please let me know.

25

Hi all, it seems like it wasn’t mentioned before, but there is a Courseware search functionality that was added Learning MFE before Redwood:
Added waffle flag state for Courseware Search
Endpoint usage implementation for Courseware Search

It’s basically relying on edx-search API endpoints that were used for legacy Course search, and it will be really nice to reuse FE when switching to Meilisearch on backend.

I agree with Braden’s arguments, about the need to replace edx-search because of problems in its architecture. However, we might need to support both with an option to use one or another for some specific use cases, e.g., an LMS courseware search, as a solution for Sumac release.

26

Thank you, @Glib_Glugovskiy, for bringing this to our attention. I want to assure you that the Courseware search functionality is already on my radar. I am actively working on ensuring its compatibility with the new abstraction package. I anticipate sharing an update on this soon.

1 Like
27

I have initiated this PR to implement the new abstraction package for the existing courseware search functionality. I believe it addresses most of the points we discussed. However, I welcome any feedback on this proposed abstraction and am open to making adjustments based on your input.

To make this PR work you are required to install openedx-search-api