docs: move comment

torrust · Feb 9, 2024 · 564df07 · 564df07
1 parent a8c2f84
commit 564df07
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/src/web/api/client/v1/client.rs b/src/web/api/client/v1/client.rs
@@ -92,8 +92,6 @@ impl Client {
     ///
     /// Will panic if the request fails.    
     pub async fn get_tags(&self) -> TextResponse {
-        // code-review: some endpoint are using plural
-        // (for instance, `get_categories`) and some singular.
         self.http_client.get("/tags", Query::empty()).await.unwrap()
     }
 

diff --git a/src/web/api/server/v1/routes.rs b/src/web/api/server/v1/routes.rs
@@ -20,6 +20,7 @@ pub const API_VERSION_URL_PREFIX: &str = "v1";
 #[allow(clippy::needless_pass_by_value)]
 pub fn router(app_data: Arc<AppData>) -> Router {
     // code-review: should we use plural for the resource prefix: `users`, `categories`, `tags`?
+    // Some endpoint are using plural (for instance, `get_categories`) and some singular.
     // See: https://stackoverflow.com/questions/6845772/should-i-use-singular-or-plural-name-convention-for-rest-resources
 
     let v1_api_routes = Router::new()