AY2425S1-CS2103T-T08-1 / tp

MIT License
2 stars 5 forks source link

Update duplicate-label-message to fit within feature freeze requirements #281

Closed chongtzezhao closed 1 week ago

chongtzezhao commented 1 week ago

Update message shown when duplicate labels are input to be a more general extension of the previous version (to include duplicate public address), while keeping the same visuals. Fixes feature-freeze break caused by #272

Before: (#272)

image

Original (before feature freeze):

image

After:

image

I.e. "After" is closer to "Original" than "Before"

codecov[bot] commented 1 week ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Files with missing lines Coverage Δ Complexity Δ
...ss/model/addresses/PublicAddressesComposition.java 74.72% <ø> (ø) 69.00 <0.00> (ø)