186506915 sort by bookmarks starred

[dennisrcao]

Fulfills the following tickets
[MultiDoc] Sort by Bookmarks (Starred)

---

Link to clue-curriculum PR
concord-consortium/clue-curriculum#142
Branch name:
186506915-sort-bookmarks-clue-repo

---

Deployed branch

https://collaborative-learning.concord.org/branch/sort-by-bookmarks-starred/?appMode=demo&demoName=dennis1&fakeClass=1&fakeUser=teacher:1&problem=1.1&unit=example-config-subtabs&curriculumBranch=sort-bookmarks-clue-repo

_
LocalHost
http://localhost:8080/?appMode=dev&demoName=dennis1&fakeClass=1&fakeUser=teacher:1&problem=1.1&unit=example-config-subtabs&curriculumBranch=sort-bookmarks-clue-repo

After PR merges - we can get rid of the curriculumBranch

[codecov]

Codecov Report

Attention: 16 lines in your changes are missing coverage. Please review.

Comparison is base (056c70b) 84.13% compared to head (e1ed075) 84.05%.

Files	Patch %	Lines
src/models/stores/sorted-documents.ts	0.00%	8 Missing and 1 partial ⚠️
src/components/document/sort-work-view.tsx	0.00%	4 Missing ⚠️
src/components/navigation/document-view.tsx	40.00%	3 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2148      +/-   ##
==========================================
- Coverage   84.13%   84.05%   -0.09%     
==========================================
  Files         680      680              
  Lines       33531    33546      +15     
  Branches     8740     8743       +3     
==========================================
- Hits        28212    28196      -16     
- Misses       5038     5067      +29     
- Partials      281      283       +2     

Flag	Coverage Δ
cypress-regression	74.84% <30.43%> (-0.10%)	⬇️
cypress-smoke	36.86% <30.43%> (-0.02%)	⬇️
jest	51.82% <26.66%> (-0.02%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[cypress]

Passing run #10558 ↗︎

0

86

4

0

0

Details:

Fixed canvas_test_spec.js and teacher_curatio_spec.js
Project: collaborative-learning	Commit: e1ed075ed6
Status: Passed	Duration: 11:48 💡
Started: Jan 24, 2024 11:51 PM	Ended: Jan 25, 2024 12:03 AM

Review all test suite changes for PR #2148 ↗︎

[scytacki]

The code looks good. There are 3 cypress tests that are failing because the class name of the sub tab was changed from "starred" to "bookmarked".

The 1st cypress test is failing because of a change to the moth unit, and that is unrelated to this PR.

[scytacki]

Looks good to me.