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

[PE-D][Tester D] Duplicate module mappings #120

Closed
nus-se-script opened this issue Apr 1, 2023 · 0 comments
Closed

[PE-D][Tester D] Duplicate module mappings #120

nus-se-script opened this issue Apr 1, 2023 · 0 comments
Assignees
@SSzeWen
Milestone
v2.1

Comments

@nus-se-script
Copy link

image.png

Steps to recreate: add ku/AMSE216

add ku/amsE216

list current

There is duplicate mapping in the list. As a ME student, i should be able to map the modules only once.

Labels: severity.Medium type.FunctionalityBug
original: geraldkoh4/ped#7
@SSzeWen SSzeWen self-assigned this Apr 1, 2023
@SSzeWen SSzeWen added this to the v2.1 milestone Apr 1, 2023
SSzeWen added a commit that referenced this issue Apr 2, 2023
@SSzeWen
Merge pull request #143 from SSzeWen/branch-fix-duplicate-module-mapp…
e72b9e4 
…ings

[Enhancement] Disallow duplicate module mappings to be saved to database Issue #120
@SSzeWen SSzeWen closed this as completed Apr 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SSzeWen SSzeWen

Labels
None yet
Projects
None yet
Milestone
v2.1
Development

No branches or pull requests
2 participants
@nus-se-script @SSzeWen