Skip to content

Fixup build errors for the web example#782

Merged
johnpryan merged 2 commits intofleaflet:masterfrom
nickoe:master
Dec 4, 2020
Merged

Fixup build errors for the web example#782
johnpryan merged 2 commits intofleaflet:masterfrom
nickoe:master

Conversation

@nickoe
Copy link
Copy Markdown
Contributor

@nickoe nickoe commented Dec 1, 2020

Tested with:

flutter build web
flutter build apk

This addresses the concerns raised in #774

Otherwise flutter throws an error and abort building.

Fixes fleaflet#774
As suggested by joselbr2099 the API have changed in flutter. Replace
only colors parameter with textColor.

Fixes fleaflet#774
@johnpryan johnpryan merged commit d6d94aa into fleaflet:master Dec 4, 2020
@johnpryan
Copy link
Copy Markdown
Collaborator

Thanks!

escamoteur pushed a commit to escamoteur/flutter_map that referenced this pull request Jan 6, 2021
* Add new required environment spec

Otherwise flutter throws an error and abort building.

Fixes fleaflet#774

* Update usage of API for text colors

As suggested by joselbr2099 the API have changed in flutter. Replace
only colors parameter with textColor.

Fixes fleaflet#774
johnpryan added a commit that referenced this pull request Jan 8, 2021
* patched version

* V0.10.2

* fix if PolylineOption.borderColor==null

* Fixup build errors for the web example (#782)

* Add new required environment spec

Otherwise flutter throws an error and abort building.

Fixes #774

* Update usage of API for text colors

As suggested by joselbr2099 the API have changed in flutter. Replace
only colors parameter with textColor.

Fixes #774

* remove mbtiles tile provider + sqflite dependency (#787)

* remove mbtiles and sqlflite dependency

* remove mbtiles tile provider + sqflite dependency

fixes #786

* remove example/test/widget_test

* updated to latest master version

* formatting code

Co-authored-by: nickoe <oe.nick@gmail.com>
Co-authored-by: John Ryan <ryjohn@google.com>
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.

2 participants