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

open release/palm.1/backport search #3

Conversation

CefBoud
Copy link

@CefBoud CefBoud commented Nov 5, 2023

Description

Backport of new Search functionnality to Palm release.

Ref: BB-8088

rijuma and others added 3 commits November 5, 2023 14:29
* feat: Added Courseware Search container popover

* chore: Added unit tests for CoursewareSearch and CoursewareSearchToggle

* chore: Updated unit test for CourseTabsNavigation

* chore: Partial coverage on Courseware Search Hooks

* chore: Finished Courseware Search Hooks unit testing

* fix: Fixed an overlook that caused a conditional hook

* fix: Reduced bounce timeout on scroll/resize to 100ms

* chore: Updated snapshots

* chore: Moved @testing-library/react-hooks dep to DEV

* chore: Minor adjustments on unit tests

* chore: Fixed test issue
* feat: Add CoursewareSearchBar component

* fix: lint

* fix: Clarified component names and i18n description

* test: Add more tests

* fix: lint

* fix: Made props in CoursewareSearchForm optional
@CefBoud CefBoud force-pushed the Cefboud/open-release/palm.1/backport-search branch from 0b7fefb to afff925 Compare November 6, 2023 15:20
* feat: Added CoursewareSearchResults UI component

* fix: Added conditional for undefined case instead of null

* fix: Updated code to avoid mutation
@CefBoud CefBoud force-pushed the Cefboud/open-release/palm.1/backport-search branch from afff925 to c9c93dc Compare November 6, 2023 15:44
Copy link

codecov bot commented Nov 6, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ No coverage uploaded for pull request base (opencraft-release/palm.1@1c025f0). Click here to learn what that means.

Additional details and impacted files
@@                     Coverage Diff                     @@
##             opencraft-release/palm.1       #3   +/-   ##
===========================================================
  Coverage                            ?   87.10%           
===========================================================
  Files                               ?      260           
  Lines                               ?     4467           
  Branches                            ?     1126           
===========================================================
  Hits                                ?     3891           
  Misses                              ?      556           
  Partials                            ?       20           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Agrendalath Agrendalath deleted the Cefboud/open-release/palm.1/backport-search branch June 10, 2024 20:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants