update comments

Author: Brian Chen

.changeset/dirty-pandas-pay.md

@@ -6,5 +6,5 @@
 This change contains multiple quality-of-life improvements when using the `FirestoreDataConverter` in `@firebase/firestore/lite` and `@firebase/firestore`:
 - Support for passing in `FieldValue` property values when using a converter (via `WithFieldValue<T>` and `PartialWithFieldValue<T>`).
 - Support for omitting properties in nested fields when performing a set operation with `{merge: true}` with a converter (via `PartialWithFieldValue<T>`).
-- [breaking] Support for typed update operations when using a converter (via the newly typed `UpdateData`). Improperly typed fields in
+- Support for typed update operations when using a converter (via the newly typed `UpdateData`). Improperly typed fields in
 update operations on typed document references will no longer compile.

packages/firestore/src/exp/snapshot.ts

@@ -91,7 +91,7 @@ export interface FirestoreDataConverter<T>
    * Firestore database). To use `set()` with `merge` and `mergeFields`,
    * `toFirestore()` must be defined with `PartialWithFieldValue<T>`.
    *
-   * The `WithFieldValue<T>` type extends `T` to also allow FieldValues such
+   * The `WithFieldValue<T>` type extends `T` to also allow FieldValues such as
    * {@link (deleteField:1)} to be used as property values.
    */
   toFirestore(modelObject: WithFieldValue<T>): DocumentData;
@@ -103,7 +103,7 @@ export interface FirestoreDataConverter<T>
    * and {@link (Transaction.set:1)} with `merge:true` or `mergeFields`.
    *
    * The `PartialWithFieldValue<T>` type extends `Partial<T>` to allow
-   * FieldValues such {@link (arrayUnion:1)} to be used as property values.
+   * FieldValues such as {@link (arrayUnion:1)} to be used as property values.
    * It also supports nested `Partial` by allowing nested fields to be
    * omitted.
    */

packages/firestore/src/lite/reference.ts

@@ -71,7 +71,7 @@ export type WithFieldValue<T> = T extends Primitive
   : Partial<T>;
 
 /**
- * Update data (for use with {@link (setDoc:1)}) that consists of field paths
+ * Update data (for use with {@link (updateDoc:1)}) that consists of field paths
  * (e.g. 'foo' or 'foo.baz') mapped to values. Fields that contain dots
  * reference nested fields within the document. FieldValues can be passed in
  * as property values.

packages/firestore/src/lite/snapshot.ts

@@ -85,7 +85,7 @@ export interface FirestoreDataConverter<T> {
    * Firestore database). Used with {@link @firebase/firestore/lite#(setDoc:1)}, {@link @firebase/firestore/lite#(WriteBatch.set:1)}
    * and {@link @firebase/firestore/lite#(Transaction.set:1)}.
    *
-   * The `WithFieldValue<T>` type extends `T` to also allow FieldValues such
+   * The `WithFieldValue<T>` type extends `T` to also allow FieldValues such as
    * {@link (deleteField:1)} to be used as property values.
    */
   toFirestore(modelObject: WithFieldValue<T>): DocumentData;
@@ -97,7 +97,7 @@ export interface FirestoreDataConverter<T> {
    * and {@link @firebase/firestore/lite#(Transaction.set:1)} with `merge:true` or `mergeFields`.
    *
    * The `PartialWithFieldValue<T>` type extends `Partial<T>` to allow
-   * FieldValues such {@link (arrayUnion:1)} to be used as property values.
+   * FieldValues such as {@link (arrayUnion:1)} to be used as property values.
    * It also supports nested `Partial` by allowing nested fields to be
    * omitted.
    */