nus-cs2103-AY2122S2 / forum

13 stars 1 forks source link

Is this considered a bug fix or adding a new feature? #263

Open ThePrevailingOne opened 2 years ago

ThePrevailingOne commented 2 years ago

Hi prof, just want to clarify, would adding an error message explaining that the company already exists in the archive be considered fixing a bug or adding a new feature? image image

damithc commented 2 years ago

@ThePrevailingOne This is not strictly a bug as the error message is correct (but not precise enough). So, making it more precise would count as an enhancement, which are not allowed. However, you can document this behavior better in the UG to reduce the impact of this problem.