wip users

aeltorio · aeltorio · commit 000ac12a588d · 2025-11-17T15:57:09.000+01:00
diff --git a/client/src/locales/base/ar-SA.json b/client/src/locales/base/ar-SA.json
@@ -270,5 +270,29 @@
   "confirm-delete-warning": "هل أنت متأكد أنك تريد حذف {{name}}؟",
   "failed-loading-user-permissions": "فشل في تحميل أذونات المستخدم",
   "error-saving-permissions": "خطأ في حفظ الأذونات",
-  "tester": "اختبار"
+  "tester": "اختبار",
+  "assign": "تعيين",
+  "assign-selected": "تعيين المحدد",
+  "assign-tester": "تعيين اختبار",
+  "assigned-successfully": "تم التعيين بنجاح",
+  "choose-tester": "اختر جهاز اختبار",
+  "confirm-unassign-selected": "هل أنت متأكد من أنك تريد إلغاء تعيين المعرفات المحددة من المختبرين؟",
+  "create-and-assign": "إنشاء وتعيين",
+  "create-tester": "إنشاء اختبار",
+  "enter-tester-name": "أدخل اسم المختبر",
+  "error-assigning-testers": "حدث خطأ في تعيين المختبرين",
+  "error-creating-tester": "حدث خطأ أثناء إنشاء جهاز الاختبار",
+  "error-removing-oauth-id": "حدث خطأ أثناء إزالة معرف OAuth",
+  "info": "معلومات",
+  "no-tester-found-for-id": "لم يتم العثور على مختبر لهذا المعرف",
+  "oauth-id-removed-successfully": "تمت إزالة معرف OAuth بنجاح",
+  "or-create-new-tester": "أو إنشاء اختبار جديد",
+  "select-tester": "اختر أحد المختبرين",
+  "selected-count": "تم تحديد {{count}}.",
+  "selected-unassigned": "لم يتم تعيين المعرفات المحددة",
+  "tester-created": "تم إنشاء جهاز الاختبار",
+  "tester-created-and-assigned": "تم إنشاء المُختبر وتعيينه",
+  "unassign": "إلغاء التعيين",
+  "unassign-selected": "إلغاء تحديد المحدد",
+  "user-already-assigned": "تم تعيين هذا المستخدم بالفعل للاختبار"
 }
diff --git a/client/src/locales/base/es-ES.json b/client/src/locales/base/es-ES.json
@@ -270,5 +270,29 @@
   "confirm-delete-warning": "¿Estás seguro de que deseas eliminar a {{name}}?",
   "failed-loading-user-permissions": "Error al cargar los permisos del usuario",
   "error-saving-permissions": "Error al guardar permisos",
-  "tester": "Ensayador"
+  "tester": "Ensayador",
+  "assign": "Asignar",
+  "assign-selected": "Asignar seleccionado",
+  "assign-tester": "Asignar probador",
+  "assigned-successfully": "Asignado exitosamente",
+  "choose-tester": "Elige un probador",
+  "confirm-unassign-selected": "¿Está seguro de que desea desasignar los ID seleccionados de sus evaluadores?",
+  "create-and-assign": "Crear y asignar",
+  "create-tester": "Crear un probador",
+  "enter-tester-name": "Ingrese el nombre del probador",
+  "error-assigning-testers": "Error al asignar probadores",
+  "error-creating-tester": "Error al crear el probador",
+  "error-removing-oauth-id": "Error al eliminar el ID de OAuth",
+  "info": "Información",
+  "no-tester-found-for-id": "No se encontró ningún probador para este ID",
+  "oauth-id-removed-successfully": "El ID de OAuth se eliminó correctamente",
+  "or-create-new-tester": "o crear un nuevo probador",
+  "select-tester": "Seleccione un probador",
+  "selected-count": "{{count}} seleccionado",
+  "selected-unassigned": "Se desasignaron los ID seleccionados",
+  "tester-created": "Probador creado",
+  "tester-created-and-assigned": "Probador creado y asignado",
+  "unassign": "Desasignar",
+  "unassign-selected": "Desasignar seleccionado",
+  "user-already-assigned": "Este usuario ya está asignado a un evaluador."
 }
diff --git a/client/src/locales/base/fr-FR.json b/client/src/locales/base/fr-FR.json
@@ -270,5 +270,29 @@
   "confirm-delete-warning": "Êtes-vous sûr de vouloir supprimer {{name}} ?",
   "failed-loading-user-permissions": "Échec du chargement des autorisations de l'utilisateur",
   "error-saving-permissions": "Erreur lors de l'enregistrement des permissions",
-  "tester": "Testeur"
+  "tester": "Testeur",
+  "assign": "Attribuer",
+  "assign-selected": "Attribuer la sélection",
+  "assign-tester": "Attribuer un testeur",
+  "assigned-successfully": "Attribué avec succès",
+  "choose-tester": "Choisissez un testeur",
+  "confirm-unassign-selected": "Êtes-vous sûr de vouloir annuler l'attribution des identifiants sélectionnés à leurs testeurs ?",
+  "create-and-assign": "Créer et attribuer",
+  "create-tester": "Créer un testeur",
+  "enter-tester-name": "Entrez le nom du testeur",
+  "error-assigning-testers": "Erreur lors de l'attribution des testeurs",
+  "error-creating-tester": "Erreur lors de la création du testeur",
+  "error-removing-oauth-id": "Erreur lors de la suppression de l'ID OAuth",
+  "info": "Informations",
+  "no-tester-found-for-id": "Aucun testeur trouvé pour cet ID",
+  "oauth-id-removed-successfully": "ID OAuth supprimé avec succès",
+  "or-create-new-tester": "ou créer un nouveau testeur",
+  "select-tester": "Sélectionnez un testeur",
+  "selected-count": "{{count}} sélectionnés",
+  "selected-unassigned": "Les identifiants sélectionnés n'ont pas été attribués",
+  "tester-created": "Testeur créé",
+  "tester-created-and-assigned": "Testeur créé et attribué",
+  "unassign": "Annuler l'attribution",
+  "unassign-selected": "Annuler l'attribution de la sélection",
+  "user-already-assigned": "Cet utilisateur est déjà affecté à un testeur"
 }
diff --git a/client/src/locales/base/he-IL.json b/client/src/locales/base/he-IL.json
@@ -270,5 +270,29 @@
   "confirm-delete-warning": "האם אתה בטוח שאתה רוצה למחוק את {{name}}?",
   "failed-loading-user-permissions": "נכשל טעינת הרשאות המשתמש",
   "error-saving-permissions": "שגיאה בשמירת ההרשאות",
-  "tester": "בּוֹחֵן"
+  "tester": "בּוֹחֵן",
+  "assign": "לְהַקְצוֹת",
+  "assign-selected": "הקצה נבחרים",
+  "assign-tester": "הקצה בוחן",
+  "assigned-successfully": "הוקצה בהצלחה",
+  "choose-tester": "בחר בודק",
+  "confirm-unassign-selected": "האם אתה בטוח שברצונך לבטל את הקצאת המזהים שנבחרו מהבודקים שלהם?",
+  "create-and-assign": "צור והקצה",
+  "create-tester": "צור בודק",
+  "enter-tester-name": "הזן את שם הבוחן",
+  "error-assigning-testers": "שגיאה בהקצאת בודקים",
+  "error-creating-tester": "שגיאה ביצירת בודק",
+  "error-removing-oauth-id": "שגיאה בהסרת מזהה OAuth",
+  "info": "מידע",
+  "no-tester-found-for-id": "לא נמצא בודק עבור מזהה זה",
+  "oauth-id-removed-successfully": "מזהה OAuth הוסר בהצלחה",
+  "or-create-new-tester": "או ליצור בודק חדש",
+  "select-tester": "בחר בודק",
+  "selected-count": "נבחרו {{count}}",
+  "selected-unassigned": "בוטלו הקצאת המזהים שנבחרו",
+  "tester-created": "הבוחן נוצר",
+  "tester-created-and-assigned": "הבוחן נוצר והוקצה",
+  "unassign": "בטל הקצאה",
+  "unassign-selected": "בטל ההקצאה שנבחרה",
+  "user-already-assigned": "משתמש זה כבר הוקצה לבוחן"
 }
diff --git a/client/src/locales/base/zh-CN.json b/client/src/locales/base/zh-CN.json
@@ -270,5 +270,29 @@
   "confirm-delete-warning": "您确定要删除 {{name}} 吗？",
   "failed-loading-user-permissions": "加载用户权限失败",
   "error-saving-permissions": "保存权限时出错",
-  "tester": "测试员"
+  "tester": "测试员",
+  "assign": "分配",
+  "assign-selected": "分配选定的",
+  "assign-tester": "分配测试人员",
+  "assigned-successfully": "分配成功",
+  "choose-tester": "选择测试仪",
+  "confirm-unassign-selected": "您确定要取消分配给测试人员的所选 ID 吗？",
+  "create-and-assign": "创建并分配",
+  "create-tester": "创建一个测试器",
+  "enter-tester-name": "输入测试者姓名",
+  "error-assigning-testers": "分配测试人员时出错",
+  "error-creating-tester": "创建测试器时出错",
+  "error-removing-oauth-id": "删除 OAuth ID 时出错",
+  "info": "信息",
+  "no-tester-found-for-id": "找不到该 ID 的测试人员",
+  "oauth-id-removed-successfully": "OAuth ID 已成功删除",
+  "or-create-new-tester": "或创建一个新的测试器",
+  "select-tester": "选择测试仪",
+  "selected-count": "已选择 {{count}} 个",
+  "selected-unassigned": "选定的 ID 未分配",
+  "tester-created": "测试人员已创建",
+  "tester-created-and-assigned": "创建并分配测试人员",
+  "unassign": "取消分配",
+  "unassign-selected": "取消分配所选内容",
+  "user-already-assigned": "该用户已分配给测试人员"
 }
diff --git a/client/src/pages/users-and-permissions.tsx b/client/src/pages/users-and-permissions.tsx
@@ -582,7 +582,7 @@ export default function UsersAndPermissionsPage() {
                                     {assignedName && u.user_id && (
                                         <Button color="warning" onPress={() => unassignId(u.user_id)}>{t('unassign')}</Button>
                                     )}
-                                    <Button color="danger" onPress={() => { setConfirmDeleteUser(u); setConfirmDeleteOpen(true); }} disabled={deletingUserId === u.user_id} isLoading={deletingUserId === u.user_id}>{t('delete')}</Button>
+                                    <Button className="ml-2" color="danger" onPress={() => { setConfirmDeleteUser(u); setConfirmDeleteOpen(true); }} disabled={deletingUserId === u.user_id} isLoading={deletingUserId === u.user_id}>{t('delete')}</Button>
                                 </TableCell>
                             </TableRow>
                         )
diff --git a/cloudflare-worker/test/basic.api.test.ts b/cloudflare-worker/test/basic.api.test.ts
@@ -197,19 +197,6 @@ describe('Feedback Flow API', () => {
     expect(response.data.ids).toContain(testerId);
   });
 
-  test('80. Should remove the OAuth ID from the tester', async () => {
-    // Delete using the tester's name
-    const response = await axios.delete(`${API_BASE_URL}/tester/ids`, { data: { name: 'TESTER', id: testerId }, headers: { 'Content-Type': 'application/json', 'Authorization': `Bearer ${AUTH0_TOKEN}` }, validateStatus: (status) => status < 500 });
-    expect(response.status).toBe(200);
-    expect(response.data.success).toBe(true);
-    // confirm the tester no longer has the id
-    const verify = await api.post('/testers', { ids: testerId });
-    expect(verify.status).toBe(200);
-    expect(verify.data.total).toBeGreaterThanOrEqual(0);
-    const matching = verify.data.data.find((t: any) => (t.ids || []).includes(testerId));
-    expect(matching).toBeUndefined();
-  });
-
   test('50. Should not add duplicate OAuth ID to the tester', async () => {
     const response = await api.post('/tester/ids', {
       name: 'TESTER',
@@ -232,6 +219,19 @@ describe('Feedback Flow API', () => {
     expect(response.data.error).toBe('Some IDs already exist in the database');
   });
 
+  test('80. Should remove the OAuth ID from the tester', async () => {
+    // Delete using the tester's name
+    const response = await axios.delete(`${API_BASE_URL}/tester/ids`, { data: { name: 'TESTER', id: testerId }, headers: { 'Content-Type': 'application/json', 'Authorization': `Bearer ${AUTH0_TOKEN}` }, validateStatus: (status) => status < 500 });
+    expect(response.status).toBe(200);
+    expect(response.data.success).toBe(true);
+    // confirm the tester no longer has the id
+    const verify = await api.post('/testers', { ids: testerId });
+    expect(verify.status).toBe(200);
+    expect(verify.data.total).toBeGreaterThanOrEqual(0);
+    const matching = verify.data.data.find((t: any) => (t.ids || []).includes(testerId));
+    expect(matching).toBeUndefined();
+  });
+
   test('70. Should filter testers by a single id using POST /testers', async () => {
     // Use the authenticated user id added to TESTER in test 40
     const response = await api.post('/testers', { ids: testerId });

Original file line number	Diff line number	Diff line change
`@@ -582,7 +582,7 @@ export default function UsersAndPermissionsPage() {`
`582`	`582`	`{assignedName && u.user_id && (`
`583`	`583`	`<Button color="warning" onPress={() => unassignId(u.user_id)}>{t('unassign')}</Button>`
`584`	`584`	`)}`
`585`		`- <Button color="danger" onPress={() => { setConfirmDeleteUser(u); setConfirmDeleteOpen(true); }} disabled={deletingUserId === u.user_id} isLoading={deletingUserId === u.user_id}>{t('delete')}</Button>`
	`585`	`+ <Button className="ml-2" color="danger" onPress={() => { setConfirmDeleteUser(u); setConfirmDeleteOpen(true); }} disabled={deletingUserId === u.user_id} isLoading={deletingUserId === u.user_id}>{t('delete')}</Button>`
`586`	`586`	`</TableCell>`
`587`	`587`	`</TableRow>`
`588`	`588`	`)`