refactor: remove notification retries

as notifications are coming from multiple data sources, if a data source is retrying, then notifications will take longer to load.
MetaMask · Jul 18, 2024 · f8f0dae · f8f0dae
1 parent 21f490f
commit f8f0dae
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/app/scripts/controllers/metamask-notifications/services/feature-announcements.ts b/app/scripts/controllers/metamask-notifications/services/feature-announcements.ts
@@ -26,7 +26,7 @@ export type ContentfulResult = {
 
 async function fetchFromContentful(
   url: string,
-  retries = 3,
+  retries = 1,
   retryDelay = 1000,
 ): Promise<ContentfulResult | null> {
   let lastError: Error | null = null;

diff --git a/app/scripts/controllers/metamask-notifications/utils/utils.ts b/app/scripts/controllers/metamask-notifications/utils/utils.ts
@@ -603,7 +603,7 @@ export async function makeApiCall<T>(
   endpoint: string,
   method: 'POST' | 'DELETE',
   body: T,
-  retries = 3,
+  retries = 1,
   retryDelay = 1000,
 ): Promise<Response> {
   const options: RequestInit = {