merge(#18): fix routing conflict in discussions controller

crowdparlay · Jan 30, 2024 · a73fa55 · a73fa55
2 parents 6b6d61a + 40467dd
commit a73fa55
Showing 1 changed file with 3 additions and 9 deletions.
diff --git a/src/CrowdParlay.Social.Api/v1/Controllers/DiscussionsController.cs b/src/CrowdParlay.Social.Api/v1/Controllers/DiscussionsController.cs
@@ -22,19 +22,13 @@ public class DiscussionsController : ControllerBase
     public async Task<DiscussionDto> GetDiscussionById([FromRoute] Guid discussionId) =>
         await _discussions.GetByIdAsync(discussionId);
 
-    /// <summary>
-    /// Returns all discussions.
-    /// </summary>
-    [HttpGet]
-    public async Task<IEnumerable<DiscussionDto>> GetAllDiscussions() =>
-        await _discussions.GetAllAsync();
-
     /// <summary>
     /// Returns all discussions created by author with the specified ID.
     /// </summary>
     [HttpGet]
-    public async Task<IEnumerable<DiscussionDto>> GetDiscussionsByAuthor([FromQuery] Guid authorId) =>
-        await _discussions.GetByAuthorAsync(authorId);
+    public async Task<IEnumerable<DiscussionDto>> GetDiscussions([FromQuery] Guid? authorId) => authorId is null
+        ? await _discussions.GetAllAsync()
+        : await _discussions.GetByAuthorAsync(authorId.Value);
 
     /// <summary>
     /// Creates a discussion.