From 35889ba122f39a55bc577c48427b298e4b60f036 Mon Sep 17 00:00:00 2001
From: Dave Horton <daveh@beachdognet.com>
Date: Mon, 21 Jun 2021 13:33:43 -0400
Subject: [PATCH] fix loop in phone numbers page

---
 src/components/pages/internal/CarriersList.js     | 2 +-
 src/components/pages/internal/PhoneNumbersList.js | 8 ++++----
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/components/pages/internal/CarriersList.js b/src/components/pages/internal/CarriersList.js
index 7ceb2be..d5bb610 100644
--- a/src/components/pages/internal/CarriersList.js
+++ b/src/components/pages/internal/CarriersList.js
@@ -98,7 +98,7 @@ const CarriersList = () => {
         console.log(err.response || err);
       }
     }
-  }, [currentServiceProvider]);
+  }, [currentServiceProvider, history, dispatch]);
 
   //=============================================================================
   // Delete sip trunk
diff --git a/src/components/pages/internal/PhoneNumbersList.js b/src/components/pages/internal/PhoneNumbersList.js
index 6e20c0e..64053a6 100644
--- a/src/components/pages/internal/PhoneNumbersList.js
+++ b/src/components/pages/internal/PhoneNumbersList.js
@@ -1,4 +1,4 @@
-import React, { useState, useEffect, useContext } from 'react';
+import React, { useState, useEffect, useContext, useCallback } from 'react';
 import axios from 'axios';
 import { useHistory } from 'react-router-dom';
 import { NotificationDispatchContext } from '../../../contexts/NotificationContext';
@@ -14,12 +14,12 @@ const PhoneNumbersList = () => {
 
   useEffect(() => {
     document.title = `Phone Number Routing | Jambonz | Open Source CPAAS`;
-  });
+  }, []);
 
   //=============================================================================
   // Get phone numbers
   //=============================================================================
-  const getPhoneNumbers = async () => {
+  const getPhoneNumbers = useCallback(async () => {
     try {
       if (!localStorage.getItem('token')) {
         history.push('/');
@@ -124,7 +124,7 @@ const PhoneNumbersList = () => {
         console.log(err.response || err);
       }
     }
-  };
+  }, [currentServiceProvider, dispatch, history]);
 
   //=============================================================================
   // Delete phone number