-
Notifications
You must be signed in to change notification settings - Fork 770
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 #6
Conversation
added some basic things will update more things asap
? |
@@ -48,6 +49,7 @@ | |||
* https://pub.dev/packages/flutter_redux | |||
* https://pub.dev/packages/flutter_mobx | |||
* https://pub.dev/packages/flutter_bloc | |||
* https://flutter.dev/docs/development/data-and-backend/state-mgmt/options // You Have Multiple choice in FLUTTER |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this is a good source, neither it is applicable in this context.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's an interesting link. I like that there are many options described, however, for junior devs it might be too much information and it might not be clear what option to go with.
@@ -93,5 +95,14 @@ | |||
* https://play.google.com/about/developer-content-policy/ | |||
* https://developer.android.com/distribute/best-practices/launch/launch-checklist | |||
|
|||
#### Animation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Although animations are nice, it is not in the roadmap. Maybe you should open an issue regarding the inclusion of this topic in the roadmap instead?
* https://flutter.dev/docs/development/ui/animations | ||
* https://flutter.dev/docs/development/ui/animations/tutorial | ||
|
||
#### You can add existing app into Flutter app |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also not directly related to the roadmap.
Overall, I don't think this is a reasonable PR. I propose closing it. |
"Animations" is a great topic that I missed in the roadmap. Most applications do not require more advanced animations than implicit ones, and I expect devs to familiarise themselves with animations as they go. Thanks for your contribution, but I don't think the "Animations" and "You can add existing app into Flutter app" sections should be added to the roadmap. The link to state management options provides too much information. I prefer to keep it more focused. |
added some basic things will update more things asap