DavinciDelta / pe

0 stars 0 forks source link

achievement and achieve does not work as intended #7

Open DavinciDelta opened 10 months ago

DavinciDelta commented 10 months ago

Steps to reproduce:

Expected:

error handling/feedback

Actual:

image.png

nus-se-script commented 10 months ago

Team's Response

Mulfunction due to incorrect file. This bug appear becauses somehow the command keyword is missing in the praser class during the pushing of my teammate's work. Sorry for making this confusion.

Below is its correct behavior:

image.png

image.png

image.png

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Achievement, viewg and achieve command not part of list of commands

Screenshot 2023-11-17 at 4.37.02 PM.png

Achievement, viewg and achieve commands are listed within the list of commands in help but cannot be accessed within the app

Furthermore, there's no way to add an achievement or goal because set on command is not available within the app


[original: nus-cs2113-AY2324S1/pe-interim#186] [original labels: type.FunctionalityBug severity.Medium]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Same problem due to missing command keyword in the praser class because a wrong file has been uploaded. Sorry for causing confusion. We indead have implemented the set function and all other goal-related functions. But somehow the command keywords are missing after someone's pulling and we did not uploade the corrected file to the teaching team even after we have corrected.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


## :question: Issue severity Team chose [`severity.High`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]