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

chore: upgrade to flutter_map 4.0.0 #4752

Merged
merged 1 commit into from
Nov 1, 2023

Conversation

monsieurtanuki
Copy link
Contributor

What

  • The upgrade to flutter_map 4.0.0 required some minor code change.

Screenshot

For the record, the new attribution display in flutter_map 4.0.0:

before clicking on the attribution icon after clicking on the attribution icon
Screenshot_1698740284 Screenshot_1698740293

Part of

Impacted files

  • knowledge_panel_world_map_card.dart: recoded with flutter_map 4.0.0 syntax
  • pubspec.lock: wtf
  • pubspec.yaml: upgraded to flutter_map 4.0.0

Impacted files:
* `knowledge_panel_world_map_card.dart`: recoded with flutter_map 4.0.0 syntax
* `pubspec.lock`: wtf
* `pubspec.yaml`: upgraded to flutter_map 4.0.0
@codecov-commenter
Copy link

codecov-commenter commented Oct 31, 2023

Codecov Report

Merging #4752 (7791719) into develop (8261f92) will increase coverage by 0.00%.
Report is 2 commits behind head on develop.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           develop   #4752   +/-   ##
=======================================
  Coverage     9.85%   9.85%           
=======================================
  Files          313     313           
  Lines        15876   15865   -11     
=======================================
  Hits          1564    1564           
+ Misses       14312   14301   -11     
Files Coverage Δ
...owledge_panels/knowledge_panel_world_map_card.dart 0.00% <0.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@teolemon teolemon merged commit b3f73c0 into openfoodfacts:develop Nov 1, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants