Skip to content

SF-3807 Store drafts in draft tab for speedy chapter navigation#3899

Open
RaymondLuong3 wants to merge 2 commits into
masterfrom
task/sf-3807-draft-tab-cache
Open

SF-3807 Store drafts in draft tab for speedy chapter navigation#3899
RaymondLuong3 wants to merge 2 commits into
masterfrom
task/sf-3807-draft-tab-cache

Conversation

@RaymondLuong3
Copy link
Copy Markdown
Collaborator

@RaymondLuong3 RaymondLuong3 commented May 20, 2026

This PR removes the getDraft method from draft handling service in favour of the getBookDraft. It also implements a cache so that getting drafts is speedier when accessing a book that is already cached. The draft tab component has been updated to use the getBookDraft method which allows navigating between chapters to require minimal load time.


This change is Reviewable

@RaymondLuong3 RaymondLuong3 added the will require testing PR should not be merged until testers confirm testing is complete label May 20, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 20, 2026

Codecov Report

❌ Patch coverage is 96.29630% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 80.91%. Comparing base (12f0d90) to head (57368bd).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...late/editor/editor-draft/editor-draft.component.ts 90.90% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3899   +/-   ##
=======================================
  Coverage   80.90%   80.91%           
=======================================
  Files         630      630           
  Lines       40677    40694   +17     
  Branches     6578     6605   +27     
=======================================
+ Hits        32909    32926   +17     
+ Misses       6741     6740    -1     
- Partials     1027     1028    +1     

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

@RaymondLuong3 RaymondLuong3 force-pushed the task/sf-3807-draft-tab-cache branch from 3ba146c to e0ab5b9 Compare May 22, 2026 22:23
@RaymondLuong3 RaymondLuong3 force-pushed the task/sf-3807-draft-tab-cache branch from e0ab5b9 to 57368bd Compare May 22, 2026 22:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

will require testing PR should not be merged until testers confirm testing is complete

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant