tricixg / pe

0 stars 0 forks source link

Year after 4 not allowed #3

Open tricixg opened 1 year ago

tricixg 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

nus-pe-bot 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.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: my suggestion may be not in scope. However, the main issue is still a feature flaw. the user should be able to input a year more than 4