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

Upgrade to sound null-safety #870

Merged
merged 62 commits into from
Jun 4, 2021
Merged

Upgrade to sound null-safety #870

merged 62 commits into from
Jun 4, 2021

Conversation

kengu
Copy link
Contributor

@kengu kengu commented Apr 14, 2021

Moves @escamoteur nullsafety-branch in PR #851 so that the community can contribute to it without blocking progress.

@kengu kengu mentioned this pull request Apr 14, 2021
lib/src/map/map.dart Outdated Show resolved Hide resolved
@johnpryan
Copy link
Collaborator

Yes, I can take a look today.

@andreandersson
Copy link
Contributor

I've resolved the comments from @ThexXTURBOXx in #919

@JaffaKetchup
Copy link
Member

Hi,
Glad and excited to see we're almost there!
Can we also merge this as part of this release?
Thanks for the great work you guys have done :)

@danielocampo2
Copy link

Thanks for all the work you're doing, is there any update on this review? Eagerly waiting for this and the other flutter_map libs to be updated.

Thanks!

@JaffaKetchup
Copy link
Member

Hi,
Just to let you guys know, I've made some more changes to the caching section that I added in my PR #913. It might now be quite complicated, however the naming of some tile providers have made it difficult, besides the fact that Flutter's own documentation contradicts itself!
I have also slightly renewed the old offline maps section to let the user know that the guide will be using TileMill which is difficult to install and has some question marks hanging over it's repository.

johnpryan and others added 2 commits June 4, 2021 07:29
* Update README.md

- Clarified all options for offline mapping
- Fixed some capitalisation inconsistencies.

* Update README.md
@@ -7,20 +7,20 @@ class GroupLayerOptions extends LayerOptions {
List<LayerOptions> group = <LayerOptions>[];

GroupLayerOptions({
Key key,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just so it isn't missed. GroupLayerOptions should have a nullable constructor. This has been solved in #921.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merged, thanks

@JaffaKetchup
Copy link
Member

JaffaKetchup commented Jun 4, 2021

Um, my #913 PR was just overwritten. Can we add this back in please? It's quite important I think, because it explains a lot about caching etc...
https://github.com/fleaflet/flutter_map/blob/dadc596e0d7c507ee500f87381872dd7b8bd765a/README.md

@johnpryan
Copy link
Collaborator

@JaffaKetchup There were merge conflicts, so I'm re-adding that manually.

@JaffaKetchup
Copy link
Member

Ok, I've linked the exact changes I made above (click Raw to copy paste) :)

@andreandersson
Copy link
Contributor

Same goes with #919 , not sure if all those have been solved yey

johnpryan and others added 6 commits June 4, 2021 08:17
* fix: remove deprecated properties

Removes deprecated properties, updates documentation and changed
bounds getCenter to a getter.

* fix: simplify center getter

* docs: improve documentation for NetworkTileProvider

* docs: update documentation for tileProvider
@johnpryan johnpryan merged commit 368e3b7 into master Jun 4, 2021
@JaffaKetchup
Copy link
Member

Hooray!
Thanks for the work everyone!

@aytunch
Copy link
Contributor

aytunch commented Jun 4, 2021

Wow :)
I am so happy. When will it be available on pub.dev? And what will be the version number?
Thanks to everyone putting in some great work.

@JaffaKetchup
Copy link
Member

Yeah same, great if this could be published to pub.dev sooner rather than later.

@johnpryan
Copy link
Collaborator

Published, thanks everyone!

@JaffaKetchup JaffaKetchup deleted the issues/829-nullsafety branch June 19, 2022 08:12
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.