physionet-build icon indicating copy to clipboard operation
physionet-build copied to clipboard

CITI training was set to expire after 1095 days (`valid_duration=datetime.timedelta(days=1095)`)

Open tompollard opened this issue 2 years ago • 2 comments

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).

tompollard avatar Apr 20 '22 18:04 tompollard

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 avatar Dec 12 '23 22:12 mscanlan-git

@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.

tompollard avatar Dec 13 '23 17:12 tompollard