Skip to content
This repository has been archived by the owner on Apr 6, 2024. It is now read-only.

[PE-D][Tester D] Room Number as the Attribute name #163

Closed
nus-pe-bot opened this issue Nov 3, 2023 · 0 comments · Fixed by #209
Closed

[PE-D][Tester D] Room Number as the Attribute name #163

nus-pe-bot opened this issue Nov 3, 2023 · 0 comments · Fixed by #209
Assignees
@limjunxian1
Labels
severity.Low A flaw that is unlikely to affect normal operations of the product. triaged.Suggestion [External] Bug reports that are suggestions rather than bugs to be fixed
Milestone
v1.4

Comments

@nus-pe-bot
Copy link

image.png

When adding any new Person, the input variable is mentioned to be address but in the display as shown in the picture, it is stored under Room Number, which is slightly unintuitive. Additionally, this might seem to cater to only those who stay on campus and on the same building where the only difference in Address is the Room Number, which might be confusing to the target audience.

Labels: type.FeatureFlaw severity.Low
original: pra-navi/ped#1
@limjunxian1 limjunxian1 self-assigned this Nov 6, 2023
@limjunxian1 limjunxian1 added severity.Low A flaw that is unlikely to affect normal operations of the product. triaged.Suggestion [External] Bug reports that are suggestions rather than bugs to be fixed labels Nov 6, 2023
@limjunxian1 limjunxian1 added this to the v1.4 milestone Nov 6, 2023
@limjunxian1 limjunxian1 mentioned this issue Nov 7, 2023
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@limjunxian1 limjunxian1

Labels
severity.Low A flaw that is unlikely to affect normal operations of the product. triaged.Suggestion [External] Bug reports that are suggestions rather than bugs to be fixed
Projects
None yet
Milestone
v1.4
Development

Successfully merging a pull request may close this issue.

Fix UserGuide.md as per PED suggestions limjunxian1/tp
2 participants
@nus-pe-bot @limjunxian1