Coverage increased (+0.2%) to 79.048% when pulling45fcb23 on meic:original-run-at intoc97f959 on arteria:master.

Thanks for the PR and sorry for the delay.

That's a good idea to track this.

While reviewing I noticedthis line. I think we should review this line first. I didn't implemented these lines myself, but viewed from afar, we should just keep the original run_at there.

I like the idea of adding original_run_at (instead of copying Task.run_at to CompletedTask.run_at as suggested by@philippeowagner), because then you can get an estimate of how long it took the task to run. I'm happy to merge conflicts here if that would be helpful to getting this merged.

Sorry for not responding sooner!

To clarify my original thoughts on this:

• run_at (as it currently stands) is useful to understand how long a background task took to execute
• original_run_at (the new field) is useful to know how long after it was scheduled to run, that it actually ran

I didn't re-use or change run_at as I didn't want to lose this information and didn't want to introduce a breaking change.

If there is anything I can do to help let me know. Otherwise is@schmave is happy to merge conflicts, that would be great!

I don't have access to modify your branch, but I merged conflicts here:https://github.com/arteria/django-background-tasks/compare/master...cmu-cs-academy:original-run-at?expand=1 so you could pull from that.