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
Updated counter example to the new @riverpod syntax #2247
Conversation
examples/counter/lib/main.dart
Outdated
@@ -20,7 +23,13 @@ class MyApp extends StatelessWidget { | |||
} | |||
|
|||
/// Providers are declared globally and specify how to create a state |
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.
The comments need to be reviewed to match the codegen syntax.
"declared globally" is more about how providers where final
global variables. But here it's a class, nothing special.
On the other hand people may want to know what that class means. In particular the build
method
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.
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.
Hum not fond of them still. I'll update them to my liking, give me a minute.
Alright~ I'll wait for the CI to pass and merge this. Thanks! |
I replaced StateProvider with a provider created with @riverpod