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

Copy new "independentAssessmentsHardLimit" field to assessments #334

Closed
crablab opened this issue Jul 30, 2020 · 5 comments
Closed

Copy new "independentAssessmentsHardLimit" field to assessments #334

crablab opened this issue Jul 30, 2020 · 5 comments
Assignees
@warrensearle
Milestone
20NOVA

Comments

@crablab
Copy link
Contributor

crablab commented Jul 30, 2020 •
edited by warrensearle

In the function where we create assessments, we should also copy over the exercise.independentAssessmentsHardLimit as assessment.hardLimit, where the former exists (optional field).
@warrensearle warrensearle mentioned this issue Aug 7, 2020
Merged
@Rebecca-mcknight1991
Copy link

@warrensearle do we still need this?

@warrensearle
Copy link
Member

@Rebecca-mcknight1991 yes this is needed if we want to show Assessors a warning about being late uploading their assessment

@Rebecca-mcknight1991
Copy link

@warrensearle ok dokey

@warrensearle
Copy link
Member

This is done

@warrensearle warrensearle added this to the 20NOVA milestone Nov 10, 2020
@crablab
Copy link
Contributor Author

crablab commented Nov 10, 2020

@Rebecca-mcknight1991 🎉 😊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@warrensearle warrensearle

Labels
None yet
Projects
None yet
Milestone
20NOVA
Development

No branches or pull requests
3 participants
@crablab @warrensearle @Rebecca-mcknight1991