mitodl / micromasters

Portal for learners and course teams to access MITx Micromasters® programs
https://mm.mit.edu
BSD 3-Clause "New" or "Revised" License
29 stars 17 forks source link

include honor certificates in grade freezing #2500

Open pdpinch opened 7 years ago

pdpinch commented 7 years ago

Early runs of MicroMasters courses (both SCM and DEDP) offered honor certificates. The course teams have confirmed that would like to count these legacy certificates for MicroMasters credit.

pdpinch commented 7 years ago

@giocalitri what are our options for doing this? I've had fewer than 5 students request this kind of update, so if there's a manual way to do it that may be acceptable. But that number could increase when we get more SCM students coming to the portal.

giocalitri commented 7 years ago

I should have already done it in this way after the conversation we had about the old courses. We need to test everything, of course.

giocalitri commented 7 years ago

Meaning this should be already in production

pdpinch commented 7 years ago

Ok. I'll assign this to myself to test then.

pdpinch commented 7 years ago

In my investigation, I haven't found any honor certificates amongst our cached certificates. Is it possible that we're not storing them from edX?

In [21]: from dashboard.models import CachedCertificate
    ...: s = set()
    ...: ct = set()
    ...: for cc in CachedCertificate.objects.all():
    ...:     s.add(cc.data['status'])
    ...:     ct.add(cc.data['certificate_type'])
    ...: print(s)
    ...: print(ct)
    ...: 
{'unverified', 'notpassing', 'downloadable', 'unavailable'}
{'verified'}
giocalitri commented 7 years ago

You are right: you can see here that we only cache the verified certificates.

Switching to all certificates can be tricky because there is a lot of logic built around the fact that the only passing certificates are the verified ones. One example