Merge pull request #61 from renderghost/fix/issue-60-qualifications-bugs

Fix #60: Qualifications page issues

Author: renderghost

lexicons/actor/biography/qualification.json

@@ -8,7 +8,7 @@
       "key": "tid",
       "record": {
         "type": "object",
-        "required": ["title", "institution", "createdAt"],
+        "required": ["title", "type", "institution", "yearAwarded", "createdAt"],
         "properties": {
           "title": {
             "type": "string",
@@ -38,10 +38,11 @@
             "maxLength": 200,
             "description": "Field of study or specialization"
           },
-          "dateAwarded": {
-            "type": "string",
-            "format": "datetime",
-            "description": "Date the qualification was awarded"
+          "yearAwarded": {
+            "type": "integer",
+            "minimum": 1900,
+            "maximum": 2100,
+            "description": "Year the qualification was awarded (YYYY format)"
           },
           "location": {
             "type": "ref",

src/app/api/about/qualifications/route.ts

@@ -16,22 +16,22 @@ export async function POST(request: NextRequest) {
     }
 
     const body = await request.json();
-    const { title, type, institution, field, dateAwarded, location } = body;
+    const { title, type, institution, field, yearAwarded, location } = body;
 
-    if (!title || !institution) {
+    if (!title || !type || !institution || !yearAwarded) {
       return NextResponse.json(
-        { error: 'Title and institution are required' },
+        { error: 'Title, type, institution, and year awarded are required' },
         { status: 400 }
       );
     }
 
     const repo = new ProfileRepository(agent);
     const rkey = await repo.createQualification({
       title,
-      type: type || 'other',
+      type,
       institution,
       field,
-      dateAwarded,
+      yearAwarded,
       location,
     });
 
@@ -68,22 +68,22 @@ export async function PUT(request: NextRequest) {
     }
 
     const body = await request.json();
-    const { title, type, institution, field, dateAwarded, location } = body;
+    const { title, type, institution, field, yearAwarded, location } = body;
 
-    if (!title || !institution) {
+    if (!title || !type || !institution || !yearAwarded) {
       return NextResponse.json(
-        { error: 'Title and institution are required' },
+        { error: 'Title, type, institution, and year awarded are required' },
         { status: 400 }
       );
     }
 
     const repo = new ProfileRepository(agent);
     await repo.updateQualification(rkey, {
       title,
-      type: type || 'other',
+      type,
       institution,
       field,
-      dateAwarded,
+      yearAwarded,
       location,
     });
 

src/app/dashboard/about/qualifications/page.tsx

@@ -127,17 +127,12 @@ export default async function QualificationsPage() {
                   </div>
                 </CardHeader>
                 <CardContent className="pt-0">
-                  <div className="flex flex-wrap items-center gap-2">
-                    {qualification.field && (
-                      <span className="text-sm text-foreground">
-                        {qualification.field}
-                      </span>
-                    )}
-                    {qualification.dateAwarded && (
-                      <span className="text-sm text-foreground">
-                        {qualification.dateAwarded}
-                      </span>
+                  <div className="flex flex-wrap items-center gap-x-2 text-sm text-foreground">
+                    {qualification.field && <span>{qualification.field}</span>}
+                    {qualification.field && qualification.yearAwarded && (
+                      <span>•</span>
                     )}
+                    {qualification.yearAwarded && <span>{qualification.yearAwarded}</span>}
                   </div>
                 </CardContent>
               </Card>