Skip to content

Conversation

letterphile
Copy link

Please merger if there is no issue. and provide some suggestions for future development too, if possible. Thank You.

@Mhdtarek
Copy link
Contributor

👍 will build and try

@SubhamTyagi
Copy link
Owner

Question class which is also generating all 4 options also needs to be updated as per the modulo operation. Current implementation is only adding/subtracting 10, 20 and 30 to wrong options which will generate invalid options for modulo operation as in modulo operation reminder must be less than divisor. So all options must be less than divisor.

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.

3 participants