rohityadavcloud commented on code in PR #7149:
URL: https://github.com/apache/cloudstack/pull/7149#discussion_r1091704338
##########
server/src/main/java/com/cloud/projects/ProjectManagerImpl.java:
##########
@@ -707,7 +712,7 @@ public void doInTransactionWithoutResult(TransactionStatus
status) throws Resour
@Override
@DB
@ActionEvent(eventType = EventTypes.EVENT_PROJECT_UPDATE, eventDescription
= "updating project", async = true)
- public Project updateProject(final long projectId, final String
displayText, final String newOwnerName, Long userId,
+ public Project updateProject(final long projectId, String name, final
String displayText, final String newOwnerName, Long userId,
Review Comment:
@rahulbcn27 can you add a unit test is possible, or at least a `Marvin` test
to update project name?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
