-
Notifications
You must be signed in to change notification settings - Fork 5
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
Fix bugs: Check duplicate task #193
Fix bugs: Check duplicate task #193
Conversation
NereusWB922
commented
Oct 30, 2023
- Check duplicate task for bulk task assignment
- Allow assign task for all students under root
close Fix bug: never check duplicate task for bulk task assignment AY2324S1-CS2103T-W15-2#191
✅ Deploy Preview for tp-preview ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov Report
@@ Coverage Diff @@
## master #193 +/- ##
============================================
- Coverage 55.37% 54.26% -1.12%
- Complexity 577 592 +15
============================================
Files 118 119 +1
Lines 2707 2779 +72
Branches 308 336 +28
============================================
+ Hits 1499 1508 +9
- Misses 1036 1075 +39
- Partials 172 196 +24
... and 1 file with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. It is essential to ensure there are no tasks added twice to a student.