remove try catch - let the exception bubble up.

This help caller to get read the NotionApiException and handle as per their need.

Author: KoditkarVedant

Src/Notion.Client/Api/Databases/DatabasesClient.cs

@@ -23,48 +23,27 @@ public DatabasesClient(IRestClient client)
 
         public async Task<Database> RetrieveAsync(string databaseId)
         {
-            try
-            {
-                return await _client.GetAsync<Database>(DatabasesApiUrls.Retrieve(databaseId));
-            }
-            catch (Exception e)
-            {
-                // Todo: Throw Custom Exception
-                return null;
-            }
+            return await _client.GetAsync<Database>(DatabasesApiUrls.Retrieve(databaseId));
         }
 
         public async Task<PaginatedList<Database>> ListAsync(DatabasesListParameters databasesListParameters = null)
         {
-            try
-            {
-                var databasesListQueryParmaters = (IDatabasesListQueryParmaters)databasesListParameters;
-                var queryParams = new Dictionary<string, string>()
-                {
-                    { "start_cursor", databasesListQueryParmaters?.StartCursor },
-                    { "page_size", databasesListQueryParmaters?.PageSize }
-                };
+            var databasesListQueryParmaters = (IDatabasesListQueryParmaters)databasesListParameters;
 
-                return await _client.GetAsync<PaginatedList<Database>>(DatabasesApiUrls.List(), queryParams);
-            }
-            catch (Exception e)
+            var queryParams = new Dictionary<string, string>()
             {
-                // Todo: Throw Custom Exception
-                return null;
-            }
+                { "start_cursor", databasesListQueryParmaters?.StartCursor },
+                { "page_size", databasesListQueryParmaters?.PageSize }
+            };
+
+            return await _client.GetAsync<PaginatedList<Database>>(DatabasesApiUrls.List(), queryParams);
         }
 
         public async Task<PaginatedList<Page>> QueryAsync(string databaseId, DatabasesQueryParameters databasesQueryParameters)
         {
-            try
-            {
-                var body = (IDatabaseQueryBodyParameters)databasesQueryParameters;
-                return await _client.PostAsync<PaginatedList<Page>>(DatabasesApiUrls.Query(databaseId), body);
-            }
-            catch (Exception e)
-            {
-                return null;
-            }
+            var body = (IDatabaseQueryBodyParameters)databasesQueryParameters;
+
+            return await _client.PostAsync<PaginatedList<Page>>(DatabasesApiUrls.Query(databaseId), body);
         }
     }
 }

Src/Notion.Client/Api/Users/UsersClient.cs

@@ -21,26 +21,12 @@ public UsersClient(IRestClient client)
 
         public async Task<User> RetrieveAsync(string userId)
         {
-            try
-            {
-                return await _client.GetAsync<User>(UsersApiUrls.Retrieve(userId));
-            }
-            catch (Exception e)
-            {
-                return null;
-            }
+            return await _client.GetAsync<User>(UsersApiUrls.Retrieve(userId));
         }
 
         public async Task<PaginatedList<User>> ListAsync()
         {
-            try
-            {
-                return await _client.GetAsync<PaginatedList<User>>(UsersApiUrls.List());
-            }
-            catch (Exception e)
-            {
-                return null;
-            }
+            return await _client.GetAsync<PaginatedList<User>>(UsersApiUrls.List());
         }
     }
 }