Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Dirty Command SideEffect #118

Closed
nus-se-bot opened this issue Apr 5, 2019 · 2 comments
Closed

Dirty Command SideEffect #118

nus-se-bot opened this issue Apr 5, 2019 · 2 comments

Comments

@nus-se-bot
Copy link

Describe the bug
Dirty command increases "Times Worn" counter.

To Reproduce
Steps to reproduce the behavior:

  1. Input"Dirty 1"
  2. TImes Worn counter increase.

Expected behavior
From UserGuide only available should change to unavailable. Times Worn should not change

Screenshots
Screenshot 2019-04-05 at 2 43 44 PM

Additional context
Add any other context about the problem here.

Reported by: @andrewscw
Severity: Medium

[original: nus-cs2103-AY1819S2/pe-dry-run#360]
@stanleymok
Copy link

Severity Low as it is documentation; how else would times worn increase but by making it dirty by wearing?

@EugeneYiren
Copy link

Updated in user guide for clarity.

EugeneYiren added a commit to EugeneYiren/main that referenced this issue Apr 6, 2019
@EugeneYiren
Update UG for issue CS2103-AY1819S2-W15-1#118
61bb905
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@nus-se-bot @stanleymok @EugeneYiren