Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes issue #1557 Changes to support switching to bigquery. #1568

Merged
merged 17 commits into from
May 6, 2024
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Removing unused imports
  • Loading branch information
jonespm authored Apr 26, 2024
commit a7a9ed8226bcda775e3c4b126bc637fc1761e9b2
3 changes: 1 addition & 2 deletions dashboard/common/db_util.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
# Some utility functions used by other classes in this project
import logging
import datetime
from typing import Any, Dict, List, Literal, TypedDict, Union
from typing import Dict, List, Literal, TypedDict, Union
from urllib.parse import quote_plus

import django
Expand All @@ -11,7 +11,6 @@
from django.conf import settings
from django.contrib.auth.models import User as DjangoUser
from django_cron.models import CronJobLog
from google.cloud import bigquery

from dashboard.models import Course, User

Expand Down
2 changes: 1 addition & 1 deletion dashboard/cron.py
Original file line number Diff line number Diff line change
Expand Up @@ -233,7 +233,7 @@ def update_canvas_resource(self):
# Select all the files for these courses
# convert int array to str array
df_attach = self.execute_bq_query(
self.queries['resource'],
self.queries['resource'],
bigquery.QueryJobConfig(query_parameters=[
bigquery.ArrayQueryParameter('course_ids', 'STRING', self.valid_locked_course_ids,),
])
Expand Down