ganhongyao / ped Goto Github PK

Steps to reproduce

1. open app with sample data (should have students tagged with "zoom")
2. execute command t/zo

Expected behaviour

• students with tags "zoom" should be matched, since it's mentioned in UG that keywords can match partially

Actual behaviour

• no match

"bellow" -> "below"

On executing command ladd 1 recurring/23 Nov 2022 date/30 jan 2022 time/0900-1100 subject/Math rates/37.50, this is the feedback:

Suggestion: Wrap the "Recurring" with parentheses and add a space?

Steps to reproduce

1. open app
2. execute command add n/Ryan Tan a/moony drive e/[email protected]
3. execute command add n/Ryan Tan a/sunny drive e/[email protected]

Expected behaviour

• Allow student to be added since although they have the same name, they may be different students.

Actual behaviour

• "This student already exists in TAB!"

Steps to reproduce

1. open app
2. assume time now 1713.
3. add a makeup lesson for any student from 1713-1714.
4. execute remind at 1715.

Expected behaviour

• since the lesson has ended as of 1715, it should no longer appear in the reminders.

The target user is not specified since it is unclear who "you" refers to.

Perhaps you may rephrase it to "private tutors"

Steps to reproduce

1. open app
2. assuming person 2 has currently a lesson 2, execute command ledit 2 2 f/100.
3. execute command paid 2 2 amt/99.99

Expected behaviour

• remaining amount = 0.01

Actual behaviour

• nothing happens, no feedback, outstanding fee not modified.
• probably an exception occurred?

When trying to add a lesson that has conflict with another existing lesson, the feedback is "This lesson clashes with an existing lesson."

Would be great if the exact clash is specified in case the user doesn't know

The constraint that a lesson must be conducted between 0800-2200 is not found in the UG.

If I have a lesson that ends today at 2000 and I have the app open at that time, the outstanding fees will not be incremented until I restart the app.

But according to the UG, it should update the fees once the lesson has ended.

Making the app update the outstanding fees while it is open may not be trivial, so may be good to state this in the UG somewhere.

Is there a specific reason why you are imposing this restriction on users? I feel like the user should have the freedom to decide when to conduct their lessons.