cachapa / firedart

A dart-native implementation of the Firebase Auth and Firestore SDKs
https://pub.dev/packages/firedart
Apache License 2.0
174 stars 62 forks source link

Convert Page<T> to Page<Document> #139

Closed nickmeinhold closed 4 months ago

nickmeinhold commented 4 months ago

As far as I can tell, Pages only represent collections of Documents so it seems that it would be clearer to remove the TypeParameter.

nickmeinhold commented 4 months ago

This doesn't make sense in hindsight, it would have to be just Page or PageOfDocuments without a type parameter and Page is nice and clear. Sorry for the noise.