Fix(crm): Apply suggestions from code review

marmelab · Aug 5, 2024 · 144df78 · 144df78
1 parent 0ac9738
commit 144df78
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 10 deletions.
diff --git a/src/contacts/useContactImport.tsx b/src/contacts/useContactImport.tsx
@@ -137,10 +137,10 @@ export function useContactImport() {
                         gender,
                         title,
                         email,
-                        phone_1_number: phoneNumber1Type,
-                        phone_1_type: phoneNumber1Number,
-                        phone_2_number: phoneNumber2Type,
-                        phone_2_type: phoneNumber2Number,
+                        phone_1_number,
+                        phone_1_type,
+                        phone_2_number,
+                        phone_2_type,
                         background,
                         acquisition,
                         first_seen,
@@ -168,10 +168,10 @@ export function useContactImport() {
                                 gender,
                                 title,
                                 email,
-                                phone_1_number: phoneNumber1Number,
-                                phone_1_type: phoneNumber1Type,
-                                phone_2_number: phoneNumber2Number,
-                                phone_2_type: phoneNumber2Type,
+                                phone_1_number,
+                                phone_1_type,
+                                phone_2_number,
+                                phone_2_type,
                                 background,
                                 acquisition,
                                 first_seen: new Date(first_seen),

diff --git a/src/dataProvider.ts b/src/dataProvider.ts
@@ -239,7 +239,7 @@ export const dataProvider = withLifecycleCallbacks(
                     'company_name',
                     'title',
                     'email',
-                    'phone_2_number',
+                    'phone_1_number',
                     'phone_2_number',
                     'background',
                 ])(params);

diff --git a/src/misc/getCompanyAvatar.ts b/src/misc/getCompanyAvatar.ts
@@ -22,7 +22,6 @@ export async function getCompanyAvatar(record: Partial<Company>): Promise<{
     src: string;
     title: string;
 } | null> {
-    console.log('Getting avatar for company', record);
     if (!record.website) {
         return null;
     }