cedvdb comments

Results 215 comments of


                                            cedvdb

trafficstars

Phone number is not supported - Help required?

The functionality isn't there yet right ?

Update merge fix

> It is not a good idea to chain PRs for unrelated issues. Agreed. I was trying to resolve a specific query issue that affected me personally and applied every...

cannot watch stream and create at the same time

fixed in https://github.com/atn832/fake_cloud_firestore/pull/220

Batch + querySnapshot gives Null check operator used on a null value

I can confirm that I still have this error from the master branch. ```yaml fake_cloud_firestore: git: url: https://github.com/atn832/fake_cloud_firestore.git ```

[Feature] Zoom in to decluster

yes your while loop is problematic because it is synchronous so count will always be the same. What you'd want is somehting like this ```dart // pseudo code int getClusterZoomLevel()...

[Feature] Zoom in to decluster

As far as I know there is no public method in the package that does it. A Pull request would certainly help to resolve this issue if that's not the...

[Feature] Zoom in to decluster

I don't think either of us are working on that, feel free to implement it.

[gen_l10n] Add option to format generated localizations files

I'm confused why this is opt-in and even optional. In what situation wouldn't you need the file to be formatted like the rest of the project ? How will this...

[gen_l10n] Add option to format generated localizations files

> 1. if you don't use dart format on your codebase > 2. if we don't make this opt-in, it will be a breaking change to customers who check in...

[gen_l10n] Add option to format generated localizations files

> Not necessarily breaking, but it would be a bit annoying/confusing having to commit unexpected changes due to formatting changes. That's only valid for the first localization gen and only...