Skip to content

Commit

Permalink
Remove comment
Browse files Browse the repository at this point in the history
I think including a message here is okay, even if it's not used
currently. It's consistent with the behaviour of the other subscribe and
unsubscribe endpoints.
  • Loading branch information
tjmw committed May 17, 2024
1 parent 9320045 commit 7deb184
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion src/server/lib/idapi/subscriptions.ts
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,6 @@ export const makeUnsubscribeAllRequest = async (
request_id,
});

// Maybe we don't return an error message here? There's nothing user facing
throw new IdapiError({ message: 'Unsubscribe all failed', status: 500 });
}
};

0 comments on commit 7deb184

Please sign in to comment.