Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Null-safety with #28 improvement #29

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Conversation

giandifra
Copy link
Owner

No description provided.

juancki and others added 12 commits April 10, 2021 11:38
Because {} depends on clustering_google_maps from git which depends on path_provider ^1.1.0, path_provider ^1.1.0 is required.
So, because lists_polls depends on path_provider ^2.0.1, version solving failed.
pub get failed (1; So, because lists_polls depends on path_provider ^2.0.1, version solving failed.)
Because every version of clustering_google_maps from git depends on google_maps_flutter ^1.0.3 and {} depends on google_maps_flutter ^2.0.1, clustering_google_maps from git is forbidden.
So, because {} depends on clustering_google_maps from git, version solving failed.
pub get failed (1; So, because lists_polls depends on clustering_google_maps from git, version solving failed.)
Error: Method not found: 'BitmapDescriptor.fromAsset'.
              BitmapDescriptor.fromAsset(bitmapAssetPathForSingleMarker);
Removed prints in debug mode
… the array of markers at init time and caching previous results based on zoom level.
@giandifra giandifra changed the title Null safety with #28 improvement Null-safety with #28 improvement Jul 30, 2021
@giandifra
Copy link
Owner Author

Hi @juancki , thank you for your contribution.
I migrate the package to null safety and I merged your PR into this.
I'd like to reintroduce this to make the update backwards compatible or adding an example to read from database.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants