Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Internship position does not accept strings with non alphanumeric characters such at C++ #1600

Open
nus-pe-bot opened this issue Nov 11, 2022 · 1 comment

Comments

@nus-pe-bot
Copy link

nus-pe-bot commented Nov 11, 2022

When adding or editing internship positions with non alphanumeric characters such as C++, the input validation will reject the string even though positions such as C++ Engineer or C# Engineer are valid and real software engineering positions. Perhaps the input validation should be relaxed fori nternship position to allow for special characters to accomodate these positions.

position not accepted


[original: nus-cs2103-AY2223S1/pe-interim#1371] [original labels: type.FeatureFlaw severity.Low]
@JustinS00
Copy link
Contributor

JustinS00 commented Nov 12, 2022

Team's Response

This is valid however such cases are rare for internship positions. User can modify the position for instance by dropping the ++ and indicating the specific language under remarks. For this iteration, internship position should only contain alphanumeric characters and spaces as stated in the User Guide and error message. Our team has therefore decided that this is not in scope.

Duplicate status (if any):

--

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants