From b958da3bc49ee37caead791b7192445784002c1d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Let=C3=ADcia=20Camara?= Date: Tue, 30 Oct 2018 21:17:51 -0300 Subject: [PATCH 01/11] Fix ambassadors and merchant sort undefined error --- src/components/EnhancedTable.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/components/EnhancedTable.js b/src/components/EnhancedTable.js index c7eb76a..a878a7a 100644 --- a/src/components/EnhancedTable.js +++ b/src/components/EnhancedTable.js @@ -94,15 +94,15 @@ class EnhancedTable extends Component { const data = order === 'desc' ? this.state.data.sort((a, b) => { - let a_value = a[orderBy]; - let b_value = b[orderBy]; + let a_value = (a[orderBy] !== undefined) ? a[orderBy]: ''; + let b_value = (b[orderBy] !== undefined) ? b[orderBy]: ''; a_value = a_value.hasOwnProperty('searchText') ? a_value.searchText.toLowerCase() : a_value.toLowerCase(); b_value = b_value.hasOwnProperty('searchText') ? b_value.searchText.toLowerCase() : b_value.toLowerCase(); return (b_value < a_value) ? -1 : 1; }) : this.state.data.sort((a, b) => { - let a_value = a[orderBy]; - let b_value = b[orderBy]; + let a_value = (a[orderBy] !== undefined) ? a[orderBy]: ''; + let b_value = (b[orderBy] !== undefined) ? b[orderBy]: ''; a_value = a_value.hasOwnProperty('searchText') ? a_value.searchText.toLowerCase() : a_value.toLowerCase(); b_value = b_value.hasOwnProperty('searchText') ? b_value.searchText.toLowerCase() : b_value.toLowerCase(); if(a_value.trim() === '') a_value = 'zzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzz'; From 3b4593540f6be5e93563a76fc79d618748885f95 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Let=C3=ADcia=20Camara?= Date: Wed, 31 Oct 2018 14:28:06 -0300 Subject: [PATCH 02/11] Remove sort link on the Map column --- src/components/EnhancedTable.js | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/components/EnhancedTable.js b/src/components/EnhancedTable.js index a878a7a..8eb820d 100644 --- a/src/components/EnhancedTable.js +++ b/src/components/EnhancedTable.js @@ -87,6 +87,8 @@ class EnhancedTable extends Component { const orderBy = property; let order = 'desc'; + if(orderBy === 'map') return; + if (this.state.orderBy === property && this.state.order === 'desc') { order = 'asc'; } From 57a63fa711a5e5f722ca427fc30229e7e3bcb8ff Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Let=C3=ADcia=20Camara?= Date: Wed, 31 Oct 2018 19:53:04 -0300 Subject: [PATCH 03/11] Replace city and country by sortable location --- src/components/pages/AmbassadorsPage.js | 2 +- src/components/pages/MerchantsPage.js | 7 +++++-- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/src/components/pages/AmbassadorsPage.js b/src/components/pages/AmbassadorsPage.js index b25c16a..9b34dd7 100644 --- a/src/components/pages/AmbassadorsPage.js +++ b/src/components/pages/AmbassadorsPage.js @@ -394,7 +394,7 @@ class AmbassadorsPage extends Component { Address: {merchant.address} {(merchant.phone) && (
Phone: {merchant.phone}
)} ; + merchant.location = { + searchText: `${merchant.country} - ${merchant.city}`, + value: (merchant.city) ? `${merchant.city} - ${merchant.country}`: merchant.country + } merchant.map =