physionet-build
physionet-build copied to clipboard
CITI training was set to expire after 1095 days (`valid_duration=datetime.timedelta(days=1095)`)
We are now able to set a "valid_duration" for different training types. We had intended to set this to None
for PhysioNet CITI training, meaning training would never expire.
After checking the database, I noticed that the valid_duration
had been set to 1095 days. This means that training may have expired for users who submitted their training ~3 years ago.
I have set the valid_duration
for CITI training to None
, so training will not expire now. I need to look into whether this has any negative downstream consequences (e.g. some users may now have two active training reports).
Looking back at some old issues, since we have changed the expiration date on CITI training reports, it would probably be a good idea to remove the 'expire' column when viewing training reports? Doesn't seem like it's necessary if a report will never expire (although, CITI reports do expire on CITI's end, so maybe not?)
@mscanlan-git I think we want to continue to display expiry dates because: (1) we are introducing support for non-CITI training courses, some of which will have expiry dates (2) ideally we would be tracking expiry dates for CITI courses too.