feat: edx endpoint API added, search options added in collection API (#1204)

* fix(deps): update dependency webpack-bundle-tracker to v3.2.0 (#1218)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* fix: Collection API updated to support search by video fields

* feat: Folder structure added for Collection page with search functionality

* test: edxEndpoint tests fixed

* test: CollectionListPage tests fixed and new tests added

* fix: Show all collections button even there is no endpoint

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

Author: marslanabdulrauf

odl_video/settings.py

@@ -72,6 +72,7 @@
     "django.contrib.messages",
     "django.contrib.staticfiles",
     "rest_framework",
+    "django_filters",
     "hijack",
     "hijack.contrib.admin",
     "encrypted_model_fields",

poetry.lock

@@ -45,6 +45,7 @@ structlog-sentry = "^1.2.2"
 urllib3 = "^1.24.2"
 uwsgi = "2.0.29"
 mitol-django-transcoding = "^2025.5.21"
+django-filter = "^25.1"
 
 
 [tool.poetry.group.dev.dependencies]

pyproject.toml

@@ -20,12 +20,25 @@ actionCreators.receiveGetPageFailure = createAction(
   constants.RECEIVE_GET_PAGE_FAILURE
 )
 
+constants.RESET_PAGES = qualifiedName("RESET_PAGES")
+actionCreators.resetPages = createAction(constants.RESET_PAGES)
+
 actionCreators.getPage = (opts: { page: number }) => {
   const { page } = opts
   const thunk = async (dispatch: Dispatch) => {
+    // Get filters from URL parameters
+    const params = new URLSearchParams(window.location.search)
+    const searchQuery = params.get('search')
+    const edxEndpointId = params.get('edx_endpoint')
+
+    // Construct query parameters
+    const queryParams = { page }
+    if (searchQuery) queryParams.search = searchQuery
+    if (edxEndpointId) queryParams.edx_endpoint = edxEndpointId
+
     dispatch(actionCreators.requestGetPage({ page }))
     try {
-      const response = await api.getCollections({ pagination: { page } })
+      const response = await api.getCollections({ pagination: queryParams })
       // @TODO: ideally we would dispatch an action here to save collections to
       // a single place in state (e.g. state.collections).
       // However, it take a non-trivial refactor to implement this schema

static/js/actions/collectionsPagination.js

@@ -0,0 +1,45 @@
+// @flow
+import { createAction } from "redux-actions"
+import type { Dispatch } from "redux"
+import * as api from "../lib/api"
+
+const qualifiedName = (name: string) => `EDX_ENDPOINTS_${name}`
+const constants = {}
+const actionCreators = {}
+
+constants.REQUEST_GET_ENDPOINTS = qualifiedName("REQUEST_GET_ENDPOINTS")
+actionCreators.requestGetEndpoints = createAction(constants.REQUEST_GET_ENDPOINTS)
+
+constants.RECEIVE_GET_ENDPOINTS_SUCCESS = qualifiedName("RECEIVE_GET_ENDPOINTS_SUCCESS")
+actionCreators.receiveGetEndpointsSuccess = createAction(
+  constants.RECEIVE_GET_ENDPOINTS_SUCCESS
+)
+
+constants.RECEIVE_GET_ENDPOINTS_FAILURE = qualifiedName("RECEIVE_GET_ENDPOINTS_FAILURE")
+actionCreators.receiveGetEndpointsFailure = createAction(
+  constants.RECEIVE_GET_ENDPOINTS_FAILURE
+)
+
+actionCreators.getEndpoints = () => {
+  const thunk = async (dispatch: Dispatch) => {
+    dispatch(actionCreators.requestGetEndpoints())
+    try {
+      const response = await api.getEdxEndpoints()
+      dispatch(
+        actionCreators.receiveGetEndpointsSuccess({
+          endpoints: response
+        })
+      )
+      return response
+    } catch (error) {
+      dispatch(
+        actionCreators.receiveGetEndpointsFailure({
+          error
+        })
+      )
+    }
+  }
+  return thunk
+}
+
+export { actionCreators, constants }

static/js/actions/edxEndpoints.js

@@ -0,0 +1,53 @@
+// @flow
+import { assert } from "chai"
+import sinon from "sinon"
+import configureTestStore from "redux-asserts"
+
+import rootReducer from "../reducers"
+import { actions } from "../actions"
+import * as api from "../lib/api"
+import { makeEdxEndpointList } from "../factories/edxEndpoints"
+import * as edxEndpointActions from "./edxEndpoints"
+
+describe("edxEndpoints actions", () => {
+  let sandbox, store, dispatchThen
+
+  beforeEach(() => {
+    sandbox = sinon.createSandbox()
+    store = configureTestStore(rootReducer)
+    dispatchThen = store.createDispatchThen(state => state.edxEndpoints)
+  })
+
+  afterEach(() => {
+    sandbox.restore()
+  })
+
+  describe("getEndpoints", () => {
+    it("fetches endpoints", async () => {
+      const endpoints = makeEdxEndpointList()
+      const getEndpointsStub = sandbox
+        .stub(api, "getEdxEndpoints")
+        .returns(Promise.resolve(endpoints))
+
+      const { data } = await dispatchThen(actions.edxEndpoints.getEndpoints(), [
+        edxEndpointActions.constants.REQUEST_GET_ENDPOINTS,
+        edxEndpointActions.constants.RECEIVE_GET_ENDPOINTS_SUCCESS
+      ])
+      assert.deepEqual(data, endpoints)
+      sinon.assert.calledWith(getEndpointsStub)
+    })
+
+    it("handles errors", async () => {
+      const errorMessage = "Error fetching endpoints"
+      sandbox
+        .stub(api, "getEdxEndpoints")
+        .returns(Promise.reject(new Error(errorMessage)))
+
+      const { error } = await dispatchThen(actions.edxEndpoints.getEndpoints(), [
+        edxEndpointActions.constants.REQUEST_GET_ENDPOINTS,
+        edxEndpointActions.constants.RECEIVE_GET_ENDPOINTS_FAILURE
+      ])
+      assert.equal(error.message, errorMessage)
+    })
+  })
+})

static/js/actions/edxEndpoints_test.js

@@ -6,11 +6,13 @@ import { endpoints } from "../lib/redux_rest"
 import * as collectionsPagination from "./collectionsPagination"
 import * as videoUi from "./videoUi"
 import * as toast from "./toast"
+import * as edxEndpoints from "./edxEndpoints"
 
 const actions: Object = {
   collectionsPagination: collectionsPagination.actionCreators,
+  edxEndpoints:          edxEndpoints.actionCreators,
   videoUi:               videoUi.actionCreators,
-  toast:                 toast.actionCreators
+  toast:                 toast.actionCreators,
 }
 
 endpoints.forEach(endpoint => {