API with transaction and write as parameter

I think transaction.GetDocumentSnapshotAsync and all similar api is not so convenient

Instead, could we write it as

firestoreDB.Collection("players").Document(follow.Key).SnapshotAsync(transaction);

It could be just an extension method

public static Task<DocumentSnapshot> SnapshotAsync(this DocumentReference doc,Transaction transaction,CancellationToken cancellationToken = default(CancellationToken))
{
    return transaction.GetDocumentSnapshotAsync(doc,cancellationToken);
}

// same for query / create / set / update / delete

Also I wish this could be possible

firestoreDB.Value.Collection("players").StreamAsync(transaction); // no equivalence in transaction

[mikelehen]

That’s a reasonable suggestion, thanks!

We’ll keep it in mind for the future and watch out for other people trying to write common code that works across Transaction and WriteBatch objects. If it’s a common pattern it may be worth introducing the interface.

[schmidt-sebastian]

Transactions are definitely not going away! There is some overlap with other APIs that might help you and others in cases where running everything within the context of a single function is not the best option.

That being said, we value user feedback tremendously and are still very much evolving our API. We haven’t gotten much feedback about this specific part of the API, and as such, value every opinion we can get on this matter. I will discuss your proposal with the rest of our team and keep you updated (likely by the end of next week).