nus-cs2103-AY2223S2 / pe-dev-response

1 stars 0 forks source link

Year after 4 not allowed #3557

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

Understand that this may be an edge case. However, this feature does not solve the stated problem of the intended user which is to aid in contact management for networking.

It is possible for students to go past the year of 4 although conventionally students graduate at year4. Hence, this input contraint placed by the software may be constriaining to the user and hinder their contact management instead of aiding.

A possible improvement would be to take in users matriculation year and month as well as expected graduation year and month. And calcualte the year from those inputs. This would greatly aid in the users contact management as the year of the contact would update yearly.

Screenshot 2023-04-14 at 2.43.55 PM.png


[original: nus-cs2103-AY2223S2/pe-interim#2844] [original labels: severity.Low type.FeatureFlaw]

anchengyang commented 1 year ago

Team's Response

Thanks for the suggestion but as the year still functions for our contact management purposes, we did not implement it as suggested. It is a good suggestion nonetheless but it would require each person to know the matriculation year as well as which year it is currently. Hence, it is a possible improvement in further iterations but not in the scope for the tp.

Duplicate status (if any):

--