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
Integrate QuizPage with the app #54
Conversation
Write functions for fetching Questions for quiz, finish quiz, save and next, previous and dispose controller
|
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.
Haven't tried running the app, but the code seems to work fine logically.
Just take care not to use absolute values for the sizes, padding and magins
@@ -1 +1,3 @@ | |||
enum CurrentState { busy, ready } | |||
|
|||
enum ChoicesEnum { optionA, optionB, optionC, optionD, NON } |
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.
Are we sure we will always have 4 options?
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 think so, not really sure
@@ -264,6 +264,8 @@ class DataEntryScreen extends GetView<DataEntryController> { | |||
width: SizeConfig.screenWidth, | |||
alignment: Alignment.center, | |||
child: CustomButton( | |||
horizontalPadding: 15, |
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.
Use relative values. SizeConfig
update(); | ||
} | ||
|
||
@override | ||
// ignore: unnecessary_overrides |
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.
Why was this override needed? is onClose( ) compulsory?
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.
Yeah, it is kinda.
@@ -117,6 +117,8 @@ class LoginView extends GetView<LoginController> { | |||
) | |||
else | |||
CustomButton( | |||
horizontalPadding: 15, |
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.
Same thing here as well. SizeConfig pls
String explanation; | ||
String? choice; |
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.
Choice can be null? when?
|
Your checklist for this pull request
🚨Please review the guidelines for contributing to this repository.
Description
Integrate QuizPage with the app using the GraphQL APIs.
❤️Thank you!
Post merge checklist
Build Status