From 983c119c78cc31f756079d6c7057f5d48cbb16b5 Mon Sep 17 00:00:00 2001 From: blam Date: Wed, 10 Jun 2020 13:57:17 +0200 Subject: [PATCH] chore(catalog/star): added a comment about why we are using a simple cache here --- plugins/catalog/src/api/CatalogClient.ts | 3 +++ 1 file changed, 3 insertions(+) diff --git a/plugins/catalog/src/api/CatalogClient.ts b/plugins/catalog/src/api/CatalogClient.ts index 9b7928a275..39e6ca8360 100644 --- a/plugins/catalog/src/api/CatalogClient.ts +++ b/plugins/catalog/src/api/CatalogClient.ts @@ -24,6 +24,9 @@ import { import Cache from 'node-cache'; export class CatalogClient implements CatalogApi { + // TODO(blam): This cache is just temporary until we have GraphQL. + // And client side caching using things like React Apollo or Relay. + // There's a lot of loading states that cause flickering around the app which aren't needed. private cache: Cache; private apiOrigin: string; private basePath: string;