Y艒ten: A social tracker for your language learning journey built on the atproto.

feat(handlers/stats): use slogger #17

merged opened by brookjeynes.dev targeting master from push-trrpxxyxxmot
Labels

None yet.

Participants 1
AT URI
at://did:plc:4mj54vc4ha3lh32ksxwunnbh/sh.tangled.repo.pull/3m3bmdlgblz22
+14 -11
Diff #0
+14 -11
internal/server/handlers/stats.go
··· 1 1 package handlers 2 2 3 3 import ( 4 - "log" 5 4 "net/http" 6 5 7 6 "yoten.app/internal/clients/bsky" ··· 12 11 ) 13 12 14 13 func (h *Handler) HandleTimePerGraphs(w http.ResponseWriter, r *http.Request) { 14 + l := h.Logger.With("handler", "HandleTimePerGraphs") 15 + 15 16 user, err := bsky.GetUserWithBskyProfile(h.Oauth, r) 16 17 if err != nil { 17 - log.Println("failed to get logged-in user:", err) 18 + l.Error("failed to get logged-in user", "err", err) 18 19 htmx.HxRedirect(w, "/login") 19 20 return 20 21 } ··· 24 25 25 26 chartData, err := db.GetTimePerData(h.Db, user.Did, period) 26 27 if err != nil { 27 - log.Println("failed to get time per chart data:", err) 28 + l.Error("failed to get time per chart data", "err", err) 28 29 chartData = db.ChartsData{ 29 30 ActivityData: []db.ChartData{}, 30 31 CategoryData: []db.ChartData{}, ··· 38 39 } 39 40 40 41 func (h *Handler) HandleStatsPage(w http.ResponseWriter, r *http.Request) { 42 + l := h.Logger.With("handler", "HandleStatsPage") 43 + 41 44 user, err := bsky.GetUserWithBskyProfile(h.Oauth, r) 42 45 if err != nil { 43 - log.Println("failed to get logged-in user:", err) 46 + l.Error("failed to get logged-in user", "err", err) 44 47 htmx.HxRedirect(w, "/login") 45 48 return 46 49 } 47 50 48 51 totalStudyTime, err := db.GetTotalStudyTime(h.Db, user.Did) 49 52 if err != nil { 50 - log.Println("failed to get total study time:", err) 53 + l.Error("failed to get total study time", "err", err) 51 54 } 52 55 53 56 totalStudySessions, err := db.GetTotalStudySessions(h.Db, user.Did) 54 57 if err != nil { 55 - log.Println("failed to get total study study sessions:", err) 58 + l.Error("failed to get total study study sessions", "err", err) 56 59 } 57 60 58 61 totalActiveDays, err := db.GetTotalActiveDays(h.Db, user.Did) 59 62 if err != nil { 60 - log.Println("failed to get total active days:", err) 63 + l.Error("failed to get total active days", "err", err) 61 64 } 62 65 63 66 streak, err := db.GetCurrentStreak(h.Db, user.Did) 64 67 if err != nil { 65 - log.Println("failed to get streak:", err) 68 + l.Error("failed to get streak", "err", err) 66 69 } 67 70 68 71 heatmap, err := db.GetHeatmapData(h.Db, user.Did) 69 72 if err != nil { 70 - log.Println("failed to get heatmap data:", err) 73 + l.Error("failed to get heatmap data", "err", err) 71 74 } 72 75 73 76 inputOutputPercentage, err := db.GetInputOutputPercentage(h.Db, user.Did) 74 77 if err != nil { 75 - log.Println("failed to get input vs output data:", err) 78 + l.Error("failed to get input vs output data", "err", err) 76 79 } 77 80 78 81 languageSummary, err := db.GetLanguageSummary(h.Db, user.Did) 79 82 if err != nil { 80 - log.Println("failed to get language time summary:", err) 83 + l.Error("failed to get language time summary", "err", err) 81 84 } 82 85 languageChartSegments := db.ConvertToDonutChartSegments(languageSummary) 83 86

History

1 round 0 comments
sign up or login to add to the discussion
brookjeynes.dev submitted #0
1 commit
expand
feat(handlers/stats): use slogger
expand 0 comments
pull request successfully merged