search

nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Edit command with no changes made is still considered valid #5697

Open nus-se-bot opened 2 years ago

nus-se-bot commented 2 years ago

Steps to replicate:

  1. addclass cn/CS2103T ct/MON 09:30-11:30 r/70 l/Nex Tuition Center
  2. editclass 1 ct/MON 09:30-11:30

This shows a success message even though no class details have been changed.

image.png

Expected:

The user should be warned that they have not made any changes to their class.

[original: nus-cs2103-AY2122S1/pe-interim#5676] [original labels: type.FunctionalityBug severity.Medium]

s7u4rt99 commented 2 years ago

Team's Response

These two issues have the same underlying problem, hence should be considered duplicates.

Duplicate status (if any):

Duplicate of #5653