Copilot commented on code in PR #11404:
URL: https://github.com/apache/cloudstack/pull/11404#discussion_r2257048384
##########
ui/src/views/infra/zone/ZoneWizardLaunchZone.vue:
##########
@@ -997,6 +997,16 @@ export default {
await
this.stepConfigurePublicTraffic('message.configuring.nsx.public.traffic',
'nsxPublicTraffic', 1)
} else if (isolationMethods.includes('netris')) {
await
this.stepConfigurePublicTraffic('message.configuring.netris.public.traffic',
'netrisPublicTraffic', 1)
+ } else {
Review Comment:
The nested if-else logic is duplicated between lines 1001-1009 and the
existing logic at lines 1012+. Consider extracting this conditional logic into
a separate method to reduce code duplication and improve maintainability.
--
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
