appview/state: handle empty vanity stat kinds

allows removing vanity stat kinds

Signed-off-by: oppiliappan <[e-mail]>

appview/db/profile.go

···
 	case models.VanityStatStarCount:
 		query = `select count(id) from stars where subject_at like 'at://' || ? || '%'`
 		args = append(args, did)
+	case models.VanityStatNone:
+		return 0, nil
 	default:
 		return 0, fmt.Errorf("invalid vanity stat kind: %s", stat)
 	}

appview/ingester.go

···
 		var stats [2]models.VanityStat
 		for i, s := range record.Stats {
 			if i < 2 {
-				stats[i].Kind = models.VanityStatKind(s)
+				stats[i].Kind = models.ParseVanityStatKind(s)
 			}
 		}
 

appview/models/profile.go

···
 	VanityStatClosedIssueCount VanityStatKind = "closed-issue-count"
 	VanityStatRepositoryCount  VanityStatKind = "repository-count"
 	VanityStatStarCount        VanityStatKind = "star-count"
+	VanityStatNone             VanityStatKind = ""
 )
 
+func ParseVanityStatKind(s string) VanityStatKind {
+	switch s {
+	case "merged-pull-request-count":
+		return VanityStatMergedPRCount
+	case "closed-pull-request-count":
+		return VanityStatClosedPRCount
+	case "open-pull-request-count":
+		return VanityStatOpenPRCount
+	case "open-issue-count":
+		return VanityStatOpenIssueCount
+	case "closed-issue-count":
+		return VanityStatClosedIssueCount
+	case "repository-count":
+		return VanityStatRepositoryCount
+	case "star-count":
+		return VanityStatStarCount
+	default:
+		return VanityStatNone
+	}
+}
+
 func (v VanityStatKind) String() string {
 	switch v {
 	case VanityStatMergedPRCount:
···
 		return "Repositories"
 	case VanityStatStarCount:
 		return "Stars Received"
+	default:
+		return ""
 	}
-	return ""
 }
 
 type VanityStat struct {

appview/pages/templates/user/fragments/editBio.html

···
   {{ $id := index . 0 }}
   {{ $stat := index . 1 }}
   <select class="stat-group w-full p-1 border border-gray-200 bg-white dark:bg-gray-800 dark:text-white dark:border-gray-700 text-sm" id="stat{{$id}}" name="stat{{$id}}">
-    <option value="">choose stat</option>
+    <option value="">Choose Stat</option>
     {{ $stats := assoc
       "merged-pull-request-count" "Merged PR Count" 
       "closed-pull-request-count" "Closed PR Count" 

appview/state/profile.go

···
 	stat0 := r.FormValue("stat0")
 	stat1 := r.FormValue("stat1")
 
-	if stat0 != "" {
-		profile.Stats[0].Kind = models.VanityStatKind(stat0)
-	}
-
-	if stat1 != "" {
-		profile.Stats[1].Kind = models.VanityStatKind(stat1)
-	}
+	profile.Stats[0].Kind = models.ParseVanityStatKind(stat0)
+	profile.Stats[1].Kind = models.ParseVanityStatKind(stat1)
 
 	if err := db.ValidateProfile(s.db, profile); err != nil {
 		log.Println("invalid profile", err)