Merge pull request #57 from techorgx/correct-httpstatus-in-response

Using appropriate http status while responding
techorgx · Dec 8, 2023 · bcb89c5 · bcb89c5
2 parents 044a215 + efb5de0
commit bcb89c5
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/main/java/com/techorgx/api/controllers/CustomerController.java b/src/main/java/com/techorgx/api/controllers/CustomerController.java
@@ -22,7 +22,7 @@ public class CustomerController {
     public ResponseEntity<String> addCustomer(@RequestBody CustomerRequestModel cm) {
         String customerId = cs.saveCustomer(cm);
         if (customerId != null) {
-            return new ResponseEntity<>(customerId, HttpStatus.ACCEPTED);
+            return new ResponseEntity<>(customerId, HttpStatus.CREATED);
         }
         return new ResponseEntity<>(HttpStatus.CONFLICT);
     }
@@ -45,7 +45,7 @@ public HttpStatus deleteCustomer(@RequestParam(name = "id") String id){
         } catch (Exception e) {
             return HttpStatus.INTERNAL_SERVER_ERROR;
         }
-        return HttpStatus.ACCEPTED;
+        return HttpStatus.OK;
     }
 
     @PutMapping(path = "/update-customer", produces = MediaType.APPLICATION_JSON_VALUE)
@@ -55,6 +55,6 @@ public HttpStatus updateCustomer(@RequestBody CustomerRequestModel cm) {
         } catch (Exception e) {
             return HttpStatus.INTERNAL_SERVER_ERROR;
         }
-        return HttpStatus.ACCEPTED;
+        return HttpStatus.OK;
     }
 }