Skip to content
This repository has been archived by the owner on Sep 9, 2022. It is now read-only.

[PE-D] Cannot add 2 internships from the same company. #146

Closed
nus-se-script opened this issue Apr 1, 2022 · 1 comment · Fixed by #191
Closed

[PE-D] Cannot add 2 internships from the same company. #146

nus-se-script opened this issue Apr 1, 2022 · 1 comment · Fixed by #191
Assignees

Comments

@nus-se-script
Copy link

A user may have 2 applications to the same company for different positions, such as Software Engineer and QA Engineer. However, InternApply only allows 1 application per company.

Use case:

  1. add n/Shopee j/Software Engineer Intern p/87438807 e/[email protected] a/5 Science Park Dr, #06-40 ast/APPLIED
  2. add n/Shopee j/QA Engineer Intern p/87438807 e/[email protected] a/5 Science Park Dr, #06-40 ast/APPLIED

Output: This application already exists in InternApply.

image.png


Labels: type.FeatureFlaw severity.Medium
original: TypeDefinition/ped#1

@lchokhoe
Copy link
Collaborator

lchokhoe commented Apr 2, 2022

"Good to fix" Bug.

Edit
Possible solution is to modify Application#isSameApplication() method. Currently uses the Application's Name as a determining factor.

Define "Duplicate Application". Suggested definition: Applications that have the same Name, Job Title, Phone, Address, Email and Optional Tags (FlexyTags) are considered duplicates.

FAQ:
I have an existing application to a company that I applied to where I got rejected and I am trying to reapply to the same company with the exact same details. Since SIA cannot have duplicate applications, what should I do?
Option 1: Using edit, you can simply reuse the existing application with updated information.

Option 2: If you are looking to keep the existing application and it's information untouched, then we suggest including an additional "t/NthAttempt" to differentiate your new application and the existing application. This will not violate our duplication clause while also allowing you to keep information on your existing application.

Related to issue #150

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

Successfully merging a pull request may close this issue.

3 participants