penn-courses icon indicating copy to clipboard operation
penn-courses copied to clipboard

PCA Resyncing Logic

Open shiva-menta opened this issue 10 months ago • 0 comments

  • Adding transaction.atomic() wrapper to webhook database calls, to reduce potential consistency errors between our StatusUpdate models and our Section model. This required modifying one of our existing test cases.
  • Modifying logic for loadstatus management command to conditionally add StatusUpdate objects to ensure consistency between the last StatusUpdate and the status attribute of the Section model. registrarimport also uses loadstatus's logic, but it currently does not add StatusUpdate objects – this can be changed easily by setting add_status_update=True.

shiva-menta avatar Apr 24 '24 03:04 shiva-menta