Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integrate Search Filters related components into lessons workflow #12788

Open
5 tasks
Tracked by #11888
marcellamaki opened this issue Nov 5, 2024 · 0 comments · May be fixed by #12871
Open
5 tasks
Tracked by #11888

Integrate Search Filters related components into lessons workflow #12788

marcellamaki opened this issue Nov 5, 2024 · 0 comments · May be fixed by #12871
Assignees
Labels
APP: Coach Re: Coach App (lessons, quizzes, groups, reports, etc.) DEV: frontend P0 - critical Priority: Release blocker or regression TODO: needs decisions Design or specifications are necessary

Comments

@marcellamaki
Copy link
Member

marcellamaki commented Nov 5, 2024

Overview

The lesson management workflow is being aligned to use the similar side panel management UX that we introduced in quizzes in 0.17. As we work on extending some of the technical aspects of that work (managing the selection of resources and folders, tracking and updating local state, etc.), we will work in parallel to build out the components and pages that surround those parts of the workflow.

This issue is to integrate the updated <SearchFiltersPanel> into lessons. The new temporary routes, added in #12672 should be used.

Prerequisites

Depends on #12521. In preparation for this issue, please add yourself as a reviewer of the PR. Catch up on the conversation, and also review the PR before you get started.

UX

Image
Image

Acceptance Criteria

  • Follow the implementation of displaying the search/filters panel within lessons, which is connected to the use of the useSearch() composable.
  • Note that in "quizzes", the activity selection is not displayed, whereas within lessons, the activity type is required
  • Do not refactor useSearch()

Optional:

Out of scope

  • Fully integrating useSearch() in lessons which requires rendering a selectable list of search results. There are interdependencies on the resource selection work being decided in parallel. Once the initial integration of the panel is completed, please check in with @marcellamaki to determine next steps about rendering cards for the search results, and what our plan will be for manual QA.

Resources

@marcellamaki marcellamaki added P0 - critical Priority: Release blocker or regression TODO: needs decisions Design or specifications are necessary APP: Coach Re: Coach App (lessons, quizzes, groups, reports, etc.) DEV: frontend labels Nov 5, 2024
@ozer550 ozer550 linked a pull request Nov 21, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
APP: Coach Re: Coach App (lessons, quizzes, groups, reports, etc.) DEV: frontend P0 - critical Priority: Release blocker or regression TODO: needs decisions Design or specifications are necessary
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants