shwstppr commented on code in PR #9752:
URL: https://github.com/apache/cloudstack/pull/9752#discussion_r2193207562


##########
api/src/main/java/org/apache/cloudstack/api/ApiConstants.java:
##########
@@ -657,7 +675,7 @@ public class ApiConstants {
     public static final String NETWORK_DEVICE_PARAMETER_LIST = 
"networkdeviceparameterlist";
     public static final String ZONE_TOKEN = "zonetoken";
     public static final String DHCP_PROVIDER = "dhcpprovider";
-    public static final String RESULT = "success";
+    public static final String RESULT = "result";

Review Comment:
   Just the constant name has been changed and the value remains same for user
   ```
   ⇒  git --no-pager grep ApiConstants.RESULT                                   
                                                                                
                                                    main|⚑32 
   
api/src/main/java/org/apache/cloudstack/api/response/CreateConsoleEndpointResponse.java:
    @SerializedName(ApiConstants.RESULT)
   
api/src/main/java/org/apache/cloudstack/api/response/RouterHealthCheckResultResponse.java:
    @SerializedName(ApiConstants.RESULT)
   
api/src/main/java/org/apache/cloudstack/api/response/UnmanageVMInstanceResponse.java:
    @SerializedName(ApiConstants.RESULT)
   ```
   
   Constant was used in 3 classes and all have been updated in
   
https://github.com/apache/cloudstack/pull/9752/commits/ed2bf1afa109a4faebfc70689968ca54cb4aa270



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to