docs: move comment

josecelano · josecelano · commit 564df072071f · 2024-02-09T16:46:57.000Z
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()

Original file line number	Diff line number	Diff line change
`@@ -92,8 +92,6 @@ impl Client {`
`92`	`92`	`///`
`93`	`93`	`/// Will panic if the request fails.`
`94`	`94`	`pub async fn get_tags(&self) -> TextResponse {`
`95`		`- // code-review: some endpoint are using plural`
`96`		- // (for instance, `get_categories`) and some singular.
`97`	`95`	`self.http_client.get("/tags", Query::empty()).await.unwrap()`
`98`	`96`	`}`
`99`	`97`