tl-its-umich-edu / my-learning-analytics

My Learning Analytics (MyLA)
Apache License 2.0
36 stars 39 forks source link

I1544 backend dependencies #1554

Closed pushyamig closed 6 months ago

pushyamig commented 7 months ago

Fixes# 1544

Changes Involved

  1. zoneinfo default timezone. migration from using pytz to using zoneinfo.
  2. Trusted Origin change: Needs an HTTPS in the configuration
  3. Cron changes are from update to SQLAlchemy V2 and psycopg V3
    1. As part of this change to postgres new driver update, you cannot make 2 queries in single format structure under the coding pattern cron.py adopted. So need to split the submission query into two.
    2. So I need to implement a different connection for this and need to run in a session. Otherwise it won't remember the temporary table all_assign_sub.
    3. django-graphene had a major upgrade as well and they removed the Promise Based Loader.
      1. I have used a new Library to resolve issues. (before it crashed Assignment planning view)
      2. At some point we need to convert all the GraphQL implementation code to adopt to this new pattern
      3. But there is an issue Open to support Promise based Loader
  4. The changes are in MyLA beta. You can access it using Canvas Beta.
    1. Ran Cron for these changes by nulling the date_last_updated .So It took about 9min to run about 56 courses.
jaydonkrooss commented 7 months ago

build was successful (after changing the is_superuser setting which was blocking me initially), and the cron job ran successfully for both courses I was testing in beta

pushyamig commented 7 months ago

@zqian and @jonespm , i would like you both review this code since it has changes to area where I might not contributed to the code like cron and GraphQL.

pushyamig commented 6 months ago

@jonespm and @zqian If this PR is not reviewed that I have to unnecessarily blocking other people working on an issue or I might endup fixing all their issue if merged before this due to library upgrade .

pushyamig commented 6 months ago

@zqian I have resolved the issues you commented on in the PR

jennlove-um commented 5 months ago

Cron runs fine and no issues detected in beta testing.