feat: remove course favorites, add program favs

backend/migrations/00020_remove_course_favorites_create_program_favorites.sql

@@ -0,0 +1,25 @@
+-- +goose Up
+-- +goose StatementBegin
+DROP TABLE IF EXISTS favorites CASCADE;
+CREATE TABLE public.program_favorites (
+	id SERIAL PRIMARY KEY,
+	program_id INT NOT NULL,
+	user_id INT NOT NULL,
+	FOREIGN KEY (program_id) REFERENCES programs(id),
+	FOREIGN KEY (user_id) REFERENCES users(id)
+);
+CREATE INDEX program_favorites_program_id_index ON program_favorites USING btree (program_id);
+CREATE INDEX program_favorites_user_id_index ON program_favorites USING btree (user_id);
+-- +goose StatementEnd
+
+-- +goose Down
+-- +goose StatementBegin
+CREATE TABLE public.favorites (
+    id SERIAL NOT NULL PRIMARY KEY,
+    user_id integer,
+    course_id integer,
+    FOREIGN KEY (user_id) REFERENCES public.users(id) ON UPDATE CASCADE ON DELETE CASCADE,
+    FOREIGN KEY (course_id) REFERENCES public.courses(id) ON UPDATE CASCADE ON DELETE CASCADE
+);
+DROP TABLE IF EXISTS program_favorites CASCADE;
+-- +goose StatementEnd

backend/src/database/DB.go

@@ -113,7 +113,7 @@ func MigrateTesting(db *gorm.DB) {
 		&models.Outcome{},
 		&models.Activity{},
 		&models.OidcClient{},
-		&models.UserFavorite{},
+		&models.ProgramFavorite{},
 		&models.Facility{},
 		&models.OpenContentProvider{},
 		&models.CronJob{},

backend/src/database/user_catalogue.go

@@ -15,16 +15,14 @@ type UserCatalogueJoin struct {
 	ExternalURL  string `json:"external_url"`
 	CourseType   string `json:"course_type"`
 	Description  string `json:"description"`
-	IsFavorited  bool   `json:"is_favorited"`
 	OutcomeTypes string `json:"outcome_types"`
 }
 
 func (db *DB) GetUserCatalogue(userId int, tags []string, search, order string) ([]UserCatalogueJoin, error) {
 	catalogue := []UserCatalogueJoin{}
 	tx := db.Table("courses c").
-		Select("c.id as course_id, c.thumbnail_url, c.name as course_name, pp.name as provider_name, c.external_url, c.type as course_type, c.description, c.outcome_types, f.user_id IS NOT NULL as is_favorited").
+		Select("c.id as course_id, c.thumbnail_url, c.name as course_name, pp.name as provider_name, c.external_url, c.type as course_type, c.description, c.outcome_types").
 		Joins("LEFT JOIN provider_platforms pp ON c.provider_platform_id = pp.id").
-		Joins("LEFT JOIN favorites f ON f.course_id = c.id AND f.user_id = ?", userId).
 		Where("c.deleted_at IS NULL").
 		Where("pp.deleted_at IS NULL")
 	for i, tag := range tags {
@@ -53,7 +51,6 @@ type UserCourses struct {
 	ProviderName   string  `json:"provider_platform_name"`
 	ExternalURL    string  `json:"external_url"`
 	CourseProgress float64 `json:"course_progress"`
-	IsFavorited    bool    `json:"is_favorited"`
 	TotalTime      uint    `json:"total_time"`
 }
 
@@ -70,7 +67,6 @@ func (db *DB) GetUserCourses(userId uint, order string, orderBy string, search s
 		"course_name":     "c.name",
 		"provider_name":   "pp.name",
 		"course_progress": "course_progress",
-		"is_favorited":    "is_favorited",
 		"total_time":      "a.total_time",
 	}
 	dbField, ok := fieldMap[orderBy]
@@ -82,7 +78,6 @@ func (db *DB) GetUserCourses(userId uint, order string, orderBy string, search s
 	tx := db.Table("courses c").
 		Select(`c.id, c.thumbnail_url,
     c.name as course_name, pp.name as provider_name, c.external_url,
-    f.user_id IS NOT NULL as is_favorited,
     CASE
         WHEN EXISTS (SELECT 1 FROM outcomes o WHERE o.course_id = c.id AND o.user_id = ?) THEN 100
         WHEN c.total_progress_milestones = 0 THEN 0
@@ -100,7 +95,6 @@ func (db *DB) GetUserCourses(userId uint, order string, orderBy string, search s
 			) a on a.course_id = c.id
         			and a.user_id = m.user_id
         			and a.RN = 1`).
-		Joins("LEFT JOIN favorites f ON f.course_id = c.id AND f.user_id = m.user_id").
 		Joins("LEFT JOIN outcomes o ON o.course_id = c.id AND o.user_id = m.user_id").
 		Where("c.deleted_at IS NULL").
 		Where("pp.deleted_at IS NULL")
@@ -113,8 +107,6 @@ func (db *DB) GetUserCourses(userId uint, order string, orderBy string, search s
 	for i, tag := range tags {
 		var query string
 		switch tag {
-		case "is_favorited":
-			query = "f.user_id IS NOT NULL"
 		case "completed":
 			query = "o.type IS NOT NULL"
 		case "in_progress":
@@ -127,7 +119,7 @@ func (db *DB) GetUserCourses(userId uint, order string, orderBy string, search s
 		}
 	}
 
-	tx.Group("c.id, c.name, c.thumbnail_url, pp.name, c.external_url, f.user_id, a.total_time")
+	tx.Group("c.id, c.name, c.thumbnail_url, pp.name, c.external_url, a.total_time")
 	err := tx.Scan(&courses).Error
 	if err != nil {
 		return nil, 0, 0, NewDBError(err, "error getting user programs")

backend/src/database/users.go

@@ -187,16 +187,16 @@ func (db *DB) UpdateUser(user *models.User) error {
 	return nil
 }
 
-func (db *DB) ToggleUserFavorite(user_id uint, id uint) (bool, error) {
+func (db *DB) ToggleProgramFavorite(user_id uint, id uint) (bool, error) {
 	var favRemoved bool
-	var favorite models.UserFavorite
-	if db.First(&favorite, "user_id = ? AND course_id = ?", user_id, id).Error == nil {
-		if err := db.Delete(&favorite).Error; err != nil {
+	var favorite models.ProgramFavorite
+	if db.First(&favorite, "user_id = ? AND program_id = ?", user_id, id).Error == nil {
+		if err := db.Unscoped().Delete(&favorite).Error; err != nil {
 			return favRemoved, newDeleteDBError(err, "favorites")
 		}
 		favRemoved = true
 	} else {
-		favorite = models.UserFavorite{UserID: user_id, CourseID: id}
+		favorite = models.ProgramFavorite{UserID: user_id, ProgramID: id}
 		if err := db.Create(&favorite).Error; err != nil {
 			return favRemoved, newCreateDBError(err, "error creating favorites")
 		}

backend/src/handlers/courses_handler.go

@@ -11,7 +11,6 @@ func (srv *Server) registerCoursesRoutes() []routeDef {
 	return []routeDef{
 		{"GET /api/courses", srv.handleIndexCourses, true, axx},
 		{"GET /api/courses/{id}", srv.handleShowCourse, false, axx},
-		{"PUT /api/courses/{id}/save", srv.handleFavoriteCourse, true, axx},
 	}
 }
 
@@ -51,23 +50,3 @@ func (srv *Server) handleShowCourse(w http.ResponseWriter, r *http.Request, log
 	}
 	return writeJsonResponse(w, http.StatusOK, course)
 }
-
-func (srv *Server) handleFavoriteCourse(w http.ResponseWriter, r *http.Request, log sLog) error {
-	id, err := strconv.Atoi(r.PathValue("id"))
-	if err != nil {
-		return newInvalidIdServiceError(err, "course ID")
-	}
-	user_id := srv.userIdFromRequest(r)
-	favoriteRemoved, err := srv.Db.ToggleUserFavorite(user_id, uint(id))
-	if err != nil {
-		log.add("course_id", id)
-		log.add("user_id", user_id)
-		return newDatabaseServiceError(err)
-	}
-	log.debugf("Favorite removed: %v", favoriteRemoved)
-	if favoriteRemoved {
-		w.WriteHeader(http.StatusNoContent)
-		return nil
-	}
-	return writeJsonResponse(w, http.StatusOK, "Favorite updated successfully")
-}

backend/src/handlers/courses_handler_test.go

@@ -6,7 +6,6 @@ import (
 	"fmt"
 	"math/rand"
 	"net/http"
-	"net/http/httptest"
 	"slices"
 	"strconv"
 	"testing"
@@ -81,41 +80,6 @@ func TestHandleShowCourse(t *testing.T) {
 	}
 }
 
-func TestHandleFavoriteCourse(t *testing.T) {
-	httpTests := []httpTest{
-		{"TestUserCanToggleFavoriteCourseOnOff", "student", map[string]any{"id": "4", "message": "Favorite updated successfully"}, http.StatusOK, ""},
-		{"TestAdminCanToggleFavoriteCourseOnOff", "admin", map[string]any{"id": "4", "message": "Favorite updated successfully"}, http.StatusOK, ""},
-	}
-	for _, test := range httpTests {
-		t.Run(test.testName, func(t *testing.T) {
-			req, err := http.NewRequest(http.MethodPut, "/api/courses/{id}/save", nil)
-			if err != nil {
-				t.Fatalf("unable to create new request, error is %v", err)
-			}
-			req.SetPathValue("id", test.mapKeyValues["id"].(string))
-			handler := getHandlerByRole(server.handleFavoriteCourse, test.role)
-			rr := executeRequest(t, req, handler, test)
-			if test.expectedStatusCode == http.StatusOK {
-				received := rr.Body.String()
-				data := models.Resource[struct{}]{}
-				if err := json.Unmarshal([]byte(received), &data); err != nil {
-					t.Errorf("failed to unmarshal response error is %v", err)
-				}
-				if data.Message != test.mapKeyValues["message"] {
-					t.Errorf("handler returned wrong body: got %v want %v", data.Message, test.mapKeyValues["message"])
-				}
-				//special case here, execute another request to toggle the user's favorite
-				req.SetPathValue("id", test.mapKeyValues["id"].(string))
-				rr = httptest.NewRecorder()
-				handler.ServeHTTP(rr, req)
-				if status := rr.Code; status != http.StatusNoContent {
-					t.Errorf("handler returned wrong status code: got %v want %v", status, http.StatusNoContent)
-				}
-			}
-		})
-	}
-}
-
 func getCourseSearch(search string) map[string]any {
 	form := make(map[string]any)
 	_, courses, err := server.Db.GetCourse(1, 10, search)

backend/src/handlers/dashboard_test.go

@@ -148,7 +148,6 @@ func TestHandleUserCourses(t *testing.T) {
 		{"TestGetUserCoursesWithTagsAndOrderByProvNmDescAsUser", "student", getUserCoursesSearch(4, "asc", "provider_name", "", []string{"certificate", "grade", "progress_completion"}), http.StatusOK, "?tags=certificate,grade,progress_completion,pathway_completion,college_credit&order=asc&order_by=provider_name"},
 		{"TestGetUserCoursesWithTagsAndOrderByCoursePgrDescAsUser", "student", getUserCoursesSearch(4, "desc", "course_progress", "", []string{"certificate", "grade", "progress_completion"}), http.StatusOK, "?tags=certificate,grade,progress_completion,pathway_completion,college_credit&order=desc&order_by=course_progress"},
 		{"TestGetUserCoursesWithTagsAndOrderByProvNmDescAsUser", "student", getUserCoursesSearch(4, "asc", "provider_name", "", []string{"certificate", "grade", "progress_completion"}), http.StatusOK, "?tags=certificate,grade,progress_completion,pathway_completion,college_credit&order=asc&order_by=provider_name"},
-		{"TestGetUserCoursesWithTagsAndOrderByIsFavdDescAsUser", "student", getUserCoursesSearch(4, "desc", "is_favorited", "", []string{"certificate", "grade", "progress_completion"}), http.StatusOK, "?tags=certificate,grade,progress_completion,pathway_completion,college_credit&order=desc&order_by=is_favorited"},
 		{"TestGetUserCoursesWithTagsAndOrderByTotalTmDescAsUser", "student", getUserCoursesSearch(4, "desc", "total_time", "", []string{"certificate", "grade", "progress_completion"}), http.StatusOK, "?tags=certificate,grade,progress_completion,pathway_completion,college_credit&order=desc&order_by=total_time"},
 		{"TestUserCoursesWithSearchAscAsUser", "student", getUserCoursesSearch(4, "asc", "", "of", []string{"certificate", "grade", "progress_completion", "pathway_completion", "college_credit"}), http.StatusOK, "?tags=certificate,grade,progress_completion,pathway_completion,college_credit&order=asc&search=of"},
 		{"TestUserCoursesWithSearchDescAsUser", "student", getUserCoursesSearch(4, "desc", "", "Intro", []string{"certificate", "grade", "progress_completion", "pathway_completion", "college_credit"}), http.StatusOK, "?tags=certificate,grade,progress_completion,pathway_completion,college_credit&order=desc&search=Intro"},

backend/src/handlers/programs_handler.go

@@ -150,7 +150,7 @@ func (srv *Server) handleFavoriteProgram(w http.ResponseWriter, r *http.Request,
 		return newInvalidIdServiceError(err, "program ID")
 	}
 	user_id := srv.userIdFromRequest(r)
-	favoriteRemoved, err := srv.Db.ToggleUserFavorite(user_id, uint(id))
+	favoriteRemoved, err := srv.Db.ToggleProgramFavorite(user_id, uint(id))
 	if err != nil {
 		log.add("programId", id)
 		log.add("userId", user_id)

backend/src/models/dashboard.go

@@ -21,12 +21,15 @@ func (LeftMenuLink) TableName() string {
 	return "left_menu_links"
 }
 
-type UserFavorite struct {
-	ID       uint `gorm:"primaryKey" json:"-"`
-	UserID   uint `json:"user_id"`
-	CourseID uint `json:"course_id"`
+type ProgramFavorite struct {
+	ID        uint `gorm:"primaryKey" json:"-"`
+	UserID    uint `json:"user_id"`
+	ProgramID uint `json:"program_id"`
+
+	User    *User    `json:"user,omitempty" gorm:"foreignKey:UserID;constraint:OnDelete CASCADE"`
+	Program *Program `json:"program,omitempty" gorm:"foreignKey:ProgramID;constraint:OnDelete CASCADE"`
 }
 
-func (UserFavorite) TableName() string {
-	return "favorites"
+func (ProgramFavorite) TableName() string {
+	return "program_favorites"
 }

backend/src/models/program.go

@@ -8,8 +8,9 @@ type Program struct {
 	ProgramStatus string `json:"program_status" gorm:"not null" validate:"required,max=50"`
 	ProgramType   string `json:"program_type" gorm:"not null" validate:"required,max=50"`
 
-	Tags       []ProgramTag `json:"tags" gorm:"foreignKey:ProgramID;references:ID"`
-	Facilities []Facility   `json:"facilities" gorm:"many2many:facilities_programs;"`
+	Tags       []ProgramTag      `json:"tags" gorm:"foreignKey:ProgramID;references:ID"`
+	Facilities []Facility        `json:"facilities" gorm:"many2many:facilities_programs;"`
+	Favorites  []ProgramFavorite `json:"-" gorm:"foreignKey:ProgramID;references:ID"`
 }
 
 func (Program) TableName() string { return "programs" }

backend/src/models/users.go

@@ -40,11 +40,11 @@ type User struct {
 	FacilityID uint     `json:"facility_id"`
 
 	/* foreign keys */
-	Mappings        []ProviderUserMapping `json:"mappings,omitempty" gorm:"foreignKey:UserID;constraint:OnDelete CASCADE"`
-	FavoriteVideos  []VideoFavorite       `json:"favorite_videos,omitempty" goem:"foreignKey:UserID;constraint:OnDelete CASCADE"`
-	FavoriteCourses []UserFavorite        `json:"favorite_courses,omitempty" gorm:"foreignKey:UserID;constraint:OnDelete CASCADE"`
-	Facility        *Facility             `json:"facility,omitempty" gorm:"foreignKey:FacilityID;constraint:OnDelete SET NULL"`
-	UserRole        *Role                 `json:"-" gorm:"foreignKey:Role;constraint:OnDelete SET NULL"`
+	Mappings         []ProviderUserMapping `json:"mappings,omitempty" gorm:"foreignKey:UserID;constraint:OnDelete CASCADE"`
+	FavoriteVideos   []VideoFavorite       `json:"favorite_videos,omitempty" goem:"foreignKey:UserID;constraint:OnDelete CASCADE"`
+	FavoritePrograms []ProgramFavorite     `json:"favorite_programs,omitempty" gorm:"foreignKey:UserID;constraint:OnDelete CASCADE"`
+	Facility         *Facility             `json:"facility,omitempty" gorm:"foreignKey:FacilityID;constraint:OnDelete SET NULL"`
+	UserRole         *Role                 `json:"-" gorm:"foreignKey:Role;constraint:OnDelete SET NULL"`
 }
 
 type ImportUser struct {

config/nginx.conf

@@ -1,75 +1,25 @@
 server {
     listen       80;
-    server_name  localhost;
+    server_name replace-me-with-host.com;
     gzip on;
 	http2 on;
 
-    location / {
-        root   /usr/share/nginx/html;
-        index  index.html index.htm;
-        try_files $uri $uri/ /index.html;
-    }
-
-	location /videos/ {
-		proxy_pass http://server:8080;
-        proxy_set_header Host $host;
-        proxy_set_header X-Real-IP $remote_addr;
-        proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
-        proxy_set_header X-Forwarded-Proto $scheme;
-    }
-
-    location /api {
-        proxy_pass http://server:8080;
-        proxy_buffering on;
-        proxy_cache_methods GET HEAD;
-        proxy_http_version 1.1;
-		proxy_set_header Real-IP $remote_addr;
-        proxy_set_header Upgrade $http_upgrade;
-        proxy_set_header Host $host;
-        proxy_cache_bypass $http_upgrade;
-    }
+    # API gateway is handled with traefik in production, so only statics are served
 
-    location /oauth2 {
-      proxy_pass http://hydra:4444;
-      proxy_http_version 1.1;
-      proxy_set_header Upgrade $http_upgrade;
-      proxy_set_header Host $host;
-      proxy_cache_bypass $http_upgrade;
+    location ~* \.(?:ico|css|js|gif|jpe?g|png|svg|woff|woff2|ttf|eot|otf|webp)$ {
+        expires 1d;
+        add_header Cache-Control "public, max-age=86400";
     }
 
-    location /sessions/ {
-      proxy_pass http://kratos:4433;
-      proxy_http_version 1.1;
-      proxy_set_header Upgrade $http_upgrade;
-      proxy_set_header Host $host;
-      proxy_cache_bypass $http_upgrade;
-      proxy_set_header X-Forwarded-Proto $scheme;
-      proxy_set_header X-Forwarded-Host $host;
+    location ~* \.(?:html)$ {
+        expires -1;
+        add_header Cache-Control "no-store, no-cache, must-revalidate, proxy-revalidate";
     }
 
-    location /self-service {
-      proxy_pass http://kratos:4433;
-      proxy_http_version 1.1;
-      proxy_set_header Upgrade $http_upgrade;
-      proxy_set_header Host $host;
-      proxy_cache_bypass $http_upgrade;
-      proxy_set_header X-Forwarded-Proto $scheme;
-      proxy_set_header X-Forwarded-Host $host;
-    }
-
-    location /.well-known/jwks.json {
-      proxy_pass http://hydra:4444;
-      proxy_http_version 1.1;
-      proxy_set_header Upgrade $http_upgrade;
-      proxy_set_header Host $host;
-      proxy_cache_bypass $http_upgrade;
-    }
-
-    location /userinfo {
-      proxy_pass http://hydra:4444;
-      proxy_http_version 1.1;
-      proxy_set_header Upgrade $http_upgrade;
-      proxy_set_header Host $host;
-      proxy_cache_bypass $http_upgrade;
-    }
+    location / {
+        root   /usr/share/nginx/html;
+        index  index.html index.htm;
+        try_files $uri $uri/ /index.html;
+        add_header Cache-Control "no-store";
+	}
 }

frontend/Dockerfile

@@ -3,7 +3,7 @@ WORKDIR /app/
 COPY . .
 RUN yarn && yarn build
 
-FROM nginx:1.21.3-alpine
+FROM nginx:1.27.2-alpine-slim
 COPY --from=builder /app/dist /usr/share/nginx/html
 EXPOSE 80
 ENTRYPOINT ["nginx", "-g", "daemon off;"]