Skip to content

SF-3925 Disable previous chapter button if chapter list is not ready - #4101

Open
pmachapman wants to merge 1 commit into
masterfrom
fix/SF-3925
Open

pmachapman wants to merge 1 commit into
masterfrom
fix/SF-3925

Conversation

@pmachapman

@pmachapman pmachapman commented Sep 14, 2026

Copy link
Copy Markdown
Collaborator

This PR ensures that the previous chapter button is disabled when the chapter list is empty.


This change is Reviewable

@pmachapman pmachapman added the will require testing PR should not be merged until testers confirm testing is complete label Sep 14, 2026
@pmachapman
pmachapman deployed to screenshot_diff September 14, 2026 00:13 — with GitHub Actions Active
@codecov

codecov Bot commented Sep 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.95%. Comparing base (568151e) to head (a53b3c0).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4101   +/-   ##
=======================================
  Coverage   80.95%   80.95%           
=======================================
  Files         664      664           
  Lines       42153    42153           
  Branches     6944     6938    -6     
=======================================
  Hits        34126    34126           
  Misses       6864     6864           
  Partials     1163     1163           

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

@Nateowami Nateowami left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

@Nateowami reviewed 1 file and all commit messages, and made 1 comment.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on pmachapman).

@Nateowami Nateowami added ready to test and removed will require testing PR should not be merged until testers confirm testing is complete labels Sep 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants