calendar_multiple dataset minor changes
Needs ReviewPublic

Authored by dekumar on Dec 28 2019, 8:35 AM.

Details

Summary

I have done the following chnages:

  1. Added a 2nd level to select date in any month in find_the date/resources/Data/1.qml
  2. Made the levels and activityConfig only visible when there is no tutorial.

Diff Detail

Repository
R2 GCompris
Branch
gsoc_akshaychd_calendar_multiple_datasets
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 20338
Build 20356: arc lint + arc unit
dekumar requested review of this revision.Dec 28 2019, 8:35 AM
dekumar created this revision.
dekumar retitled this revision from multiple dataset, proof of concept to calendar_multiple dataset minor chnages.Dec 28 2019, 8:38 AM
dekumar edited the summary of this revision. (Show Details)
dekumar added a reviewer: jjazeix.
dekumar retitled this revision from calendar_multiple dataset minor chnages to calendar_multiple dataset minor changes.

git apply D26252.diff
error: patch failed: src/activities/calendar/Calendar.qml:459
error: src/activities/calendar/Calendar.qml: patch does not apply
error: src/activities/find_the_day/resource/1/Data.qml: No such file or directory

Could you please provide a diff that works correctly from a fresh multiple_dataset branch when we apply it from directory gcompris?

git apply D26252.diff
error: patch failed: src/activities/calendar/Calendar.qml:459
error: src/activities/calendar/Calendar.qml: patch does not apply
error: src/activities/find_the_day/resource/1/Data.qml: No such file or directory

Could you please provide a diff that works correctly from a fresh multiple_dataset branch when we apply it from directory gcompris?

@echarruau Could you please try applying this diff to gsoc_akshaychd_calendar_multiple_dataset branch? As I have created a diff from this branch not from multiple_datasets.

I do not understand why we have a tutorial about leap year in this activity.

Could you do a sum up list of the changes you have done so that I can check them without forgetting anything?
Thanks.

Level 2 and 3 do not have any ok button. We will need them to be able to asses the pupils answers.

Could you do a sum up list of the changes you have done so that I can check them without forgetting anything?
Thanks.

I have added in the summary about the changes I have done.

Level 2 and 3 do not have any ok button. We will need them to be able to asses the pupils answers.

Could you please specify for which activity are you talking calendar or find_the day and for which difficulty level so that I can check and make the changes?
Thanks!

dekumar edited the summary of this revision. (Show Details)Dec 28 2019, 4:33 PM

I do not understand why we have a tutorial about leap year in this activity.

It was created by akshay earlier. Do you want me to remove it?

I will see with Johnny.

Le sam. 28 déc. 2019 à 17:35, Deepak Kumar
<noreply@phabricator.kde.org> a écrit :

dekumar added a comment. View Revision

In D26252#583959, @echarruau wrote:

I do not understand why we have a tutorial about leap year in this activity.

It was created by akshay earlier. Do you want me to remove it?

REPOSITORY
R2 GCompris

REVISION DETAIL
https://phabricator.kde.org/D26252

To: dekumar, jjazeix
Cc: echarruau, kde-edu, narvaez, apol

I will see with Johnny.

Le sam. 28 déc. 2019 à 17:35, Deepak Kumar
<noreply@phabricator.kde.org> a écrit :

dekumar added a comment. View Revision

In D26252#583959, @echarruau wrote:

I do not understand why we have a tutorial about leap year in this activity.

It was created by akshay earlier. Do you want me to remove it?

REPOSITORY
R2 GCompris

REVISION DETAIL
https://phabricator.kde.org/D26252

To: dekumar, jjazeix
Cc: echarruau, kde-edu, narvaez, apol

Okay no problem. Let me know after discussion.

@jjazeix Can you please review this once? I am looking forward to implement multiple dataset to other activities too.
Thanks :)

diff looks good, but the branch can't be merged in this state, the tutorials need to be improved

diff looks good, but the branch can't be merged in this state, the tutorials need to be improved

@jjazeix What are the changes that needs to be done to the tutorial? I can work on it for now.
Thanks!