Status: Needs Review
Changes Done:
- Added multiple datasets to Calendar and find the day activity.
- Created a generic algorithm to generate template questions as per the parameters it is provided
Status: Needs Review
Changes Done:
Apply the diff on multiple_dataset branch or pull the changes from the branch gsoc_akshaychd_calendar_multiple_datasets
No Linters Available |
No Unit Test Coverage |
Buildable 20145 | |
Build 20163: arc lint + arc unit |
src/activities/calendar/calendar.js | ||
---|---|---|
113 | Indentation | |
125 | Shouldn't Math.floor((maxMonth - minMonth)) be Math.floor(maxMonth - minMonth + 1)? Let's say minMonth is 3 and maxMonth is 5. I should get any random among 0, 1, 2 so that minMonth + Math.floor(Math.random() * Math.floor((maxMonth - minMonth))) can be anything between 3, 4, 5. But, Math.floor(Math.random() * Math.floor((maxMonth - minMonth))) limits the values among 0, 1 (as minMonth - maxMonth = 2 in our case). So you should probably do maxMonth - minMonth + 1? | |
128 | Same comment about choosing random value as above. |
I tested the patch on multiple_dataset branch, and noticed several issues.
In Find the date:
-some questions, the answer can not be entered (it is out of the accessible range)
Reproduced with first dataset at level 3 and 5 (see screenshots 1, 2 and 3)
-a question was twice the same, happened with second dataset at first level.
-some question did not accept the correct answer (see on screenshot 4 and 5, the answer to the question is correct and still the answer is not accepted).
In Calendar activity:
-At level 3, the questions mention the month and year which is not visible. Better do like in level 4 and say "of given month" instead.
Another note: the datasets have difficulty stars 1 and 3, while the activities initial difficulty level were 1 and 3 red stars (meaning level 4 and 6 respectively)
It would be better to adapt the stars of the datasets accordingly.
@timotheegiet Thanks for the review, I've updated the diff trying to resolve all the issues, can you try it out once again :)
Thanks, I tested again and can not replicate those issues anymore.
Good for me, waiting for @scagarwal and @amankumargupta to confirm it's ok for them too.
Changes done:
Thanks !
@dekumar it seems you didn't created the branch from Akshay's one, so the diff contains a lot of files that shouldn't be there.
The diff should only contain files from calendar and find_the_day activities.
To do so, checkout the good branch, and do a git checkout -b "your_branch" and update the diff here please
In D22266#581303, @jjazeix wrote:@dekumar it seems you didn't created the branch from Akshay's one, so the diff contains a lot of files that shouldn't be there.
The diff should only contain files from calendar and find_the_day activities.
To do so, checkout the good branch, and do a git checkout -b "your_branch" and update the diff here please
I have created a new branch after applying the patch still it contains a lot of files. I would update it again once.
@jjazeix Hi, I have tried again by creating a new branch it is still updating all the files.
I have made changes to find_the_day/resources/Data/1.qml and made the level,activityConfig visible only when there is no tutorial.
Commit | Tree | Parents | Author | Summary | Date |
---|---|---|---|---|---|
782e1c831582 | bc223be7d54a | adc1b619c0c1 | Deepak Kumar | levels added and indent fixed | Dec 22 2019, 11:42 AM |
adc1b619c0c1 | 9c02d652de52 | 646ee865a3ba | Deepak Kumar | Add multiple datasets to calendar activity (Show More…) | Dec 22 2019, 11:35 AM |