dev -> main (#517)

* add migrations (#514)

* fix everything blowing up when old major projects exist (#515)

---------

Co-authored-by: Tyler Allen <tyler@tallen.me>
Co-authored-by: Noah Hanford <bigspaceships56@gmail.com>

Author: 3 people

Dockerfile

@@ -19,12 +19,10 @@ RUN npm run webpack
 FROM docker.io/python:3.12-slim-bookworm
 MAINTAINER Computer Science House <webmaster@csh.rit.edu>
 
-RUN mkdir /opt/conditional
+WORKDIR /opt/conditional
 
 COPY requirements.txt /opt/conditional
 
-WORKDIR /opt/conditional
-
 RUN apt-get -yq update && \
     apt-get -yq install libsasl2-dev libldap2-dev libldap-common libssl-dev gcc g++ make && \
     pip install -r requirements.txt && \
@@ -35,6 +33,7 @@ ENV PORT=${PORT}
 EXPOSE ${PORT}
 
 COPY conditional /opt/conditional/conditional
+COPY migrations /opt/conditional/migrations
 COPY *.py package.json /opt/conditional/
 COPY --from=build-frontend /opt/conditional/conditional/static/ /opt/conditional/conditional/static/
 

conditional/blueprints/dashboard.py

@@ -92,7 +92,7 @@ def display_dashboard(user_dict=None):
             "time_spent": p.time_spent,
             "skills": p.skills,
             "desc": p.description,
-            "links": list(filter(None, p.links.split("\n"))),
+            "links": [] if p.links is None else list(filter(None, p.links.split("\n"))),
             "status": p.status,
         }
         for p in get_project_list().filter(MajorProject.uid == uid)

conditional/blueprints/major_project_submission.py

@@ -53,7 +53,7 @@ def display_major_project(user_dict=None):
             "time_spent": p.time_spent,
             "skills": p.skills,
             "desc": p.description,
-            "links": list(filter(None, p.links.split("\n"))),
+            "links": [] if p.links is None else list(filter(None, p.links.split("\n"))),
             "status": p.status,
             "is_owner": bool(user_dict["username"] == p.uid),
             "files": list_files_in_folder(bucket, f"{p.id}/")