-
Notifications
You must be signed in to change notification settings - Fork 535
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge branch 'main' into praggarg/remove-deprecated-summaryprotocol
- Loading branch information
Showing
76 changed files
with
1,776 additions
and
486 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,41 @@ | ||
--- | ||
"@fluidframework/merge-tree": minor | ||
"@fluidframework/sequence": minor | ||
--- | ||
--- | ||
"section": deprecation | ||
--- | ||
|
||
Merge-Tree and SharedString ISegment Deprecations | ||
|
||
The current ISegment interface over-exposes a number of properties which do not have an external use case, and any external usage could result in damage to the underlying merge-tree including data corruption. | ||
|
||
The only use case that will continue to be supported is determining if a segment is removed. For this purpose we've added the free function `segmentIsRemoved(segment: ISegment): boolean`. | ||
|
||
For example, checking if a segment is not removed would change as follows: | ||
``` diff | ||
- if(segment.removedSeq === undefined){ | ||
+ if(!segmentIsRemoved(segment)){ | ||
``` | ||
|
||
The following properties are deprecated on ISegment and its implementations: | ||
- clientId | ||
- index | ||
- localMovedSeq | ||
- localRefs | ||
- localRemovedSeq | ||
- localSeq | ||
- movedClientsIds | ||
- movedSeq | ||
- movedSeqs | ||
- ordinal | ||
- removedClientIds | ||
- removedSeq | ||
- seq | ||
- wasMovedOnInsert | ||
|
||
Additionally, the following types are also deprecated, and will become internal (i.e. users of the Fluid Framework will not have access to them): | ||
- IMergeNodeCommon | ||
- IMoveInfo | ||
- IRemovalInfo | ||
- LocalReferenceCollection |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.