-
Notifications
You must be signed in to change notification settings - Fork 4
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
Feature: Top Up Success (kids-844) #767
Conversation
@@ -67,7 +69,17 @@ class ChildDetailsPage extends StatelessWidget { | |||
builder: (_) => const TopUpFailureDialog(), | |||
); | |||
} else if (state is ChildTopUpSuccessState) { | |||
// TODO Kids-844 | |||
Navigator.push( |
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.
Nice 👌
@@ -0,0 +1,9 @@ | |||
class TopUpSuccessUIModel { |
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.
Good work in implementing the UIModel. The only thing I would change is I would remove the "onClickButton" here as it isn't really "UI" but rather "logic". I would pass it as a parameter in the TopUpSuccessPage itself. That way when we use UIModels in our cubits the cubits don't need to know/ define the onClick.
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.
Looks good, one small comment!
Description