Merge pull request #53 from gentlementlegen/chore/new-assign-design

manifest.json

@@ -12,4 +12,4 @@
       "description": "Unassign yourself from the issue."
     }
   }
-}
+}

src/handlers/shared/generate-assignment-comment.ts

@@ -31,11 +31,10 @@ export async function generateAssignmentComment(context: Context, issueCreatedAt
     registeredWallet:
       (await context.adapters.supabase.user.getWalletByUserId(senderId, issueNumber)) ||
       "Register your wallet address using the following slash command: `/wallet 0x0000...0000`",
-    tips: `<h6>Tips:</h6>
-    <ul>
-    <li>Use <code>/wallet 0x0000...0000</code> if you want to update your registered payment wallet address.</li>
-    <li>Be sure to open a draft pull request as soon as possible to communicate updates on your progress.</li>
-    <li>Be sure to provide timely updates to us when requested, or you will be automatically unassigned from the task.</li>
-    <ul>`,
+    tips: `
+> [!TIP]
+> - Use <code>/wallet 0x0000...0000</code> if you want to update your registered payment wallet address.
+> - Be sure to open a draft pull request as soon as possible to communicate updates on your progress.
+> - Be sure to provide timely updates to us when requested, or you will be automatically unassigned from the task.`,
   };
 }

src/handlers/shared/table.ts

@@ -1,25 +1,22 @@
 export function assignTableComment({ taskDeadline, registeredWallet, isTaskStale, daysElapsedSinceTaskCreation }: AssignTableCommentParams) {
-  let taskStaleWarning = ``;
+  const elements: string[] = ["<samp>", "<table>"];
+
   if (isTaskStale) {
-    taskStaleWarning = `<tr><td>Warning!</td> <td>This task was created over ${daysElapsedSinceTaskCreation} days ago. Please confirm that this issue specification is accurate before starting.</td></tr>`;
+    elements.push(
+      "<tr>",
+      "<td>Warning!</td>",
+      `<td><This task was created over ${daysElapsedSinceTaskCreation} days ago. Please confirm that this issue specification is accurate before starting.</td>`,
+      "</tr>"
+    );
   }
-  let deadlineWarning = ``;
+
   if (taskDeadline) {
-    deadlineWarning = `<tr><td>Deadline</td><td>${taskDeadline}</td></tr>`;
+    elements.push("<tr>", "<td>Deadline</td>", `<td>${taskDeadline}</td>`, "</tr>");
   }
 
-  return `
-<samp>
-<table>
-${taskStaleWarning}
-${deadlineWarning}
-<tr>
-<td>Beneficiary</td>
-<td>${registeredWallet}</td>
-</tr>
-</table>
-</samp>
-  `;
+  elements.push("<tr>", "<td>Beneficiary</td>", `<td>${registeredWallet}</td>`, "</tr>", "</table>", "</samp>");
+
+  return elements.join("\n");
 }
 
 interface AssignTableCommentParams {

tests/main.test.ts

@@ -246,7 +246,7 @@ describe("User start/stop", () => {
         errorDetails.push(`${error.path}: ${error.message}`);
       }
 
-      expect(errorDetails).toContain("/BOT_USER_ID: Required property");
+      expect(errorDetails).toContain("/BOT_USER_ID: Expected union value");
     }
   });