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

Update README.md #9

Closed
wants to merge 1 commit into from
Closed

Update README.md #9

wants to merge 1 commit into from

Conversation

divinepaul
Copy link

proposal to update the dataMap.
because it was confusing and and induced errors.

proposal to update the dataMap
because it was confusing and and induced errors
@apgapg
Copy link
Owner

apgapg commented Jun 24, 2019

What issue are you getting? I don't see any plus point in writing .toDouble() as my map is declared as <String,double>, hence it shouldn't create an issue

@divinepaul
Copy link
Author

What issue are you getting? I don't see any plus point in writing .toDouble() as my map is declared as <String,double>, hence it shouldn't create an issue

......well the .toDouble was the soulution to error that I got while putting a int in to your map in the documentation.

@apgapg
Copy link
Owner

apgapg commented Jun 25, 2019

I guess the problem is in the dart version. I remember the new dart accepts 4 instead of 4.0 in padding etc, they said the integers are automatically casted. Could you please check your dart version?

@divinepaul
Copy link
Author

I guess the problem is in the dart version. I remember the new dart accepts 4 instead of 4.0 in padding etc, they said the integers are automatically casted. Could you please check your dart version?


Flutter 1.5.4-hotfix.2 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 7a4c33425d (8 weeks ago) • 2019-04-29 11:05:24 -0700
Engine • revision 52c7a1e849
Tools • Dart 2.3.0 (build 2.3.0-dev.0.5 a1668566e5)

its Dart 2.3.0

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