jishnu28 / pe

0 stars 0 forks source link

Vague naming #9

Open jishnu28 opened 1 year ago

jishnu28 commented 1 year ago

Screenshot 2023-04-14 at 3.05.58 PM.png

Note could be renamed to skills if that is the only attribute that this field is intended to be used, to store and display.

nus-pe-bot commented 1 year ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Perhaps skill/ should be used instead of note/, to make it more intuitive for users

No details provided by bug reporter.


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

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

We mentioned in the note field in the Placeholder table, a note refers to the skill set of an applicant.

We agree that renaming the note field to skill could grant greater clarity. We can explore this for future iterations.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


## :question: Issue response Team chose [`response.NotInScope`] - [x] I disagree **Reason for disagreement:** The issue is not out of scope, it's just a poor naming choice that is related to features delivered in v1.4