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.

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

Issue #71: Add number of attempts to Assessment dashboard - dayview #89

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

guillogo
Copy link
Contributor

@guillogo guillogo commented Mar 3, 2021

No description provided.

@mattporritt
Copy link

Hi @guillogo
Everything you have done here is good and is exactly what I wanted in the spec.
Testing this has reminded me of something we should do, that is unfortunately going to be a reasonable amount more work. For the other assessment types that are not quiz, we should also add support for the number of attempts there too. This will be a little hard because of the different way each of the core assessments store their attempts in the database. However, there should be some code you can copy around this as it it's a similar pattern to getting start and end dates. etc.
Cheers,
Matt p

@dmitriim
Copy link
Member

dmitriim commented Dec 2, 2024

Hi @guillogo
If you still keen to get this patch in, please update your PR so it's aiming the latest MOODLE_403_STABLE? Thanks!

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

Successfully merging this pull request may close these issues.

3 participants