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.

Sign up for GitHub

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

Jump to bottom

Bug in Add Interviewee command parser #60

Closed
CornCobs opened this issue Mar 17, 2020 · 0 comments · Fixed by #65
Closed

Bug in Add Interviewee command parser #60

CornCobs opened this issue Mar 17, 2020 · 0 comments · Fixed by #65
Assignees
@agnesnatasya
Labels
priority.High Must do severity.High A flaw that affects most users and causes major problems for users. type.Bug
Milestone
v1.2

Comments

@CornCobs
Copy link

The current add interviewee command captures the entire argument as the full name of the Interviewee
Command:
add interviewee Henry Tan -a Hen

Expected: new Interviewee with name "Henry Tan" and alias "Hen"

Outcome: Interviewee with name "Henry Tan -a Hen" and no alias
@CornCobs CornCobs added type.Bug priority.High Must do severity.High A flaw that affects most users and causes major problems for users. labels Mar 17, 2020
@CornCobs CornCobs added this to the v1.2 milestone Mar 17, 2020
@agnesnatasya agnesnatasya linked a pull request Mar 17, 2020 that will close this issue
Test initialisation command parser #65
Merged
@nus-se-bot nus-se-bot mentioned this issue Mar 17, 2020
Open
26 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@agnesnatasya agnesnatasya

Labels
priority.High Must do severity.High A flaw that affects most users and causes major problems for users. type.Bug
Projects
None yet
Milestone
v1.2
Development

Successfully merging a pull request may close this issue.

Test initialisation command parser agnesnatasya/main
2 participants
@CornCobs @agnesnatasya