search

openedx / frontend-app-authoring

Front-end for the Open edX Studio experience, implemented in React and Paragon.
GNU Affero General Public License v3.0
13 stars 75 forks source link

Collections Page - Bare bones #1101

Closed jmakowski1123 closed 1 month ago

jmakowski1123 commented 5 months ago

Designs: Mid-Fi Figma link

✅ In Scope:

Spec:

  1. Users can create a collection in a Library.
  2. Each collection has a Collection Page.
  3. Users can create as many Collections in a library as they would like.
  4. When a user has created a new collection via the "new collection" button, the collection page displays as empty state.

Image

  1. When a user has created a new collection via the "add to collection" button on a content card or on a sidebar, then the collection page displays with the content populated. Content within the collection should be auto-sorted by most recently modified.

Image

Requirements:

  1. On the Collection Page, display the Collection name and an (i) button that does nothing (it's for the sidebar, in a future story)
  2. Display breadcrumbs that a user can use to navigate back to the All Collections Page or the Library Home.
  3. Include the keyword search field. https://github.com/openedx/frontend-app-course-authoring/issues/1103
  4. Include a filter for tags (https://github.com/openedx/frontend-app-course-authoring/issues/1169))
  5. Include a filter for content type (https://github.com/openedx/frontend-app-course-authoring/issues/1168)
  6. Include sort button (https://github.com/openedx/frontend-app-course-authoring/issues/1166)

Out of scope:

  1. Button to view content in list view
pomegranited commented 2 months ago

Use the object_key field added by https://github.com/openedx/modular-learning/issues/228 to identify the collection in the URL.

navinkarkera commented 2 months ago

@pomegranited Since we already have library-key in the url, we just need to include the collection key (not the whole opaque key) in the url. Discussion can be found in https://github.com/openedx/edx-platform/pull/35321#discussion_r1751972484

pomegranited commented 2 months ago

@navinkarkera Oh yes that's correct, thank you!

navinkarkera commented 1 month ago

@jmakowski1123 @lizc577 @sdaitzman @marcotuts This is ready for AC testing on the sandbox

jmakowski1123 commented 1 month ago

Collection page looks good!

I realize I missed a set of requirements for opening the collections page from other pathways - sorry about that. Created a story here: https://github.com/openedx/frontend-app-authoring/issues/1322