FluidFramework icon indicating copy to clipboard operation
FluidFramework copied to clipboard

improvement(container-runtime): Further simplifications for dirty/saved logic

Open markfields opened this issue 6 months ago • 1 comments

Description

Some follow-ups to #24646:

  • Don't bother calling updateDocumentDirtyState when processing a non-modern-runtime message, since we don't even check for its presence in PendingStateManager
  • Get rid of the special treatment of dirty state in replayPendingStates. See commit message on feef9a1fc1014281cf78bd0d663c19e2c4685d57 for details

TODO:

  • [ ] write one or two tests

markfields avatar May 23 '25 18:05 markfields

Converting to a draft, not expecting to prioritize this before going on Leave. Will finish the tests and pick it up in August.

markfields avatar Jun 06 '25 19:06 markfields

This PR has been automatically marked as stale because it has had no activity for 60 days. It will be closed if no further activity occurs within 8 days of this comment. Thank you for your contributions to Fluid Framework!