Merge pull request #1559 from RoboSats/fix-bugs-after-libraries-updates

Fix bugs after libraries updates

Author: KoalaSat

frontend/package-lock.json

@@ -92,7 +92,7 @@ const BookTable = ({
 
   const { t } = useTranslation();
   const theme = useTheme();
-  const orders = orderList ?? Object.values(federation.book);
+  const orders = orderList ?? Object.values(federation.book) ?? [];
 
   const [paginationModel, setPaginationModel] = useState<GridPaginationModel>({
     pageSize: 0,

frontend/src/components/BookTable/index.tsx

@@ -59,12 +59,9 @@ export const FederationContextProvider = ({
     if (client !== 'mobile' || torStatus === 'ON' || !settings.useProxy) {
       void federation.updateUrl(origin, settings, hostUrl);
       void federation.loadLimits();
+      federation.setConnection(settings);
     }
-  }, [settings.network, settings.useProxy, torStatus]);
-
-  useEffect(() => {
-    federation.setConnection(settings);
-  }, [settings.connection]);
+  }, [settings.network, settings.useProxy, torStatus, settings.connection]);
 
   const addNewCoordinator: (alias: string, url: string) => void = (alias, url) => {
     if (!federation.coordinators[alias]) {
@@ -90,7 +87,6 @@ export const FederationContextProvider = ({
         setCoordinatorUpdatedAt(new Date().toISOString());
       });
       garage.syncCoordinator(federation, alias);
-      setSortedCoordinators(federationLottery(federation));
       setFederationUpdatedAt(new Date().toISOString());
     }
   };

frontend/src/contexts/FederationContext.tsx

@@ -80,7 +80,6 @@ export class Federation {
 
   setConnection = (settings: Settings): void => {
     this.connection = settings.connection;
-
     if (this.connection === 'nostr') {
       this.roboPool.connect();
       this.loadBookNostr();

frontend/src/models/Federation.model.ts

@@ -70,6 +70,8 @@ const filterOrders = function ({
     .filter((coord) => coord.enabled)
     .map((coord) => coord.shortAlias);
   const filteredOrders = Object.values(federation.book).filter((order) => {
+    if (!order) return false;
+
     const coordinatorCheck = enabledCoordinators.includes(order.coordinatorShortAlias ?? '');
     const typeChecks = order.type === baseFilter.type || baseFilter.type == null;
     const modeChecks = baseFilter.mode === 'fiat' ? !(order.currency === 1000) : true;