Programming maze branch diff
ClosedPublic

Authored by amankumargupta on Dec 12 2018, 11:33 AM.

Details

Reviewers
jjazeix
Group Reviewers
GCompris: Improvements
Maniphest Tasks
Restricted Maniphest Task
Summary

This diff file contains all the work done by me (@amankumargupta ) during Season of KDE 2018.

Diff Detail

Repository
R2 GCompris
Lint
Lint Skipped
Unit
Unit Tests Skipped
Restricted Application added a project: KDE Edu. · View Herald TranscriptDec 12 2018, 11:33 AM
Restricted Application added a subscriber: kde-edu. · View Herald Transcript
amankumargupta requested review of this revision.Dec 12 2018, 11:33 AM
amankumargupta added a task: Restricted Maniphest Task.

@echarruau , @timotheegiet : is it good for you to merge it?

src/activities/programmingMaze/ActivityInfo.qml
27

we need to add the translator helpers (//: ...)

src/activities/programmingMaze/CMakeLists.txt
1

it would be better to put the instructions inside resource like for other activities

amankumargupta marked 2 inline comments as done.Jan 13 2019, 6:20 PM
amankumargupta added inline comments.
src/activities/programmingMaze/ActivityInfo.qml
27

Pushed to the branch.

Checked both behaviour and code. Beautiful piece of code :) Code is easy to understand, functions are well named, factorisation is well used everywhere it can be. Well done, congratulation!
Ready to be merged :)

amankumargupta marked an inline comment as done.Feb 12 2019, 6:21 AM

That's nice to hear! :)

jjazeix accepted this revision.Apr 27 2019, 4:19 PM
This revision is now accepted and ready to land.Apr 27 2019, 4:19 PM