search

nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Better suggestion for `better email validation` #4780

Open nus-se-script opened 4 months ago

nus-se-script commented 4 months ago

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.

image.png

Description

Since the TA ToolKit current targeted audience is for NUS CS2100 student, it would be better for TA to include @u.nus.edu emails only.

Idea

Instead of following IETF standards, can narrow down the scope to the NUS standard email.

This better helps TA to obtain the correct email to send important module informations. We would want to minimize the possibility of TA sending important school email to student's junk email or sort, causing the student to miss out on them.

[original: nus-cs2103-AY2324S2/pe-interim#4217] [original labels: type.DocumentationBug severity.VeryLow]

g-tejas commented 4 months ago

Team's Response

Hi, Thank you for your kind feedback! We intentionally did not set overly strict email constraints because allowing such flexibility can in turn allow the user to use the software in different ways while overzealous rejection of inputs can annoy the user. For example, some TAs might wish to contact with their students via other domain names, for e.g, axelteo@comp.nus.edu.sg

Duplicate status (if any):

--