Closed xie-dongping closed 8 years ago
Thank you for the pull request. Let me review it today.
The code is high quality. I have just a few small comments. Once you fixed those, I can merge you code. Good works!
@helloworld1 Thanks for the code review! I have just fixed most of the points. It just that two points require abit of clarification.
When they are clarified, I would push and squash the commit.
Replied the comments. Looking forward to a new commit and I will merge :)
Done.
Implemented the statistics function for new cards learned per day:
firstLearnDate
in the learning_data tablefirstLearnDate
in scheduler (all the old cards would have a defaultfirstLeanDate
at 2010)