-
-
Notifications
You must be signed in to change notification settings - Fork 72
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
Remove 'evaluated_by' and 'discarded' fields from TrainingProgress #2416
Remove 'evaluated_by' and 'discarded' fields from TrainingProgress #2416
Conversation
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.
Looks good to me, I left only 1 small comment.
amy/trainings/views.py
Outdated
@@ -134,31 +124,16 @@ def all_trainees(request): | |||
if request.method == "POST" and "discard" in request.POST: | |||
# Bulk discard progress of selected trainees | |||
form = BulkAddTrainingProgressForm() |
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.
Should these 3 lines remain? We could probably leave the "elif" from L149 as "if" and skip the check for "discard" in POST since that form was removed.
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.
Good point. I'll make this change.
Hey @elichad I was looking over git log history and I noticed that this PR was merged into a feature branch. Do you want to merge these changes into development branch, too? |
@pbanaszkiewicz my plan is to merge the checkout work into this |
Fixes #2409
Fixes #2407
Discussed with instructor training team and agreed that the very few existing TrainingProgress records which are currently marked as discarded will be otherwise unchanged (so some will become valid passes). Manually confirmed in AMY that this will not make anyone eligible for an instructor badge who wasn't eligible before.