Skip to content

SF-2722 Send blank ops created when the user comes online to ShareDB#3935

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

SF-2722 Send blank ops created when the user comes online to ShareDB#3935
pmachapman wants to merge 1 commit into
masterfrom
fix/SF-2722

Conversation

@pmachapman

@pmachapman pmachapman commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

This PR where verse numbers disappear when coming online then editing a text by ensuring that an blank ops created by fixSegments() make their way to ShareDB.


This change is Reviewable

@pmachapman pmachapman added the will require testing PR should not be merged until testers confirm testing is complete label Jun 10, 2026
@pmachapman pmachapman temporarily deployed to screenshot_diff June 10, 2026 01:54 — with GitHub Actions Inactive
@codecov

codecov Bot commented Jun 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.67%. Comparing base (eec50dc) to head (516d283).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3935      +/-   ##
==========================================
+ Coverage   80.66%   80.67%   +0.01%     
==========================================
  Files         634      634              
  Lines       41090    41092       +2     
  Branches     6681     6682       +1     
==========================================
+ Hits        33144    33150       +6     
+ Misses       6893     6890       -3     
+ Partials     1053     1052       -1     

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

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