Copilot commented on code in PR #13769:
URL: https://github.com/apache/cloudstack/pull/13769#discussion_r3701582011
##########
ui/src/views/infra/UsageRecords.vue:
##########
@@ -598,7 +598,7 @@ export default {
if (values.dateRange) {
Review Comment:
`values.dateRange` is always truthy because the form initializes it as `[]`
and `handleRemoveFields` doesn’t strip empty arrays. This means `getParams`
will compute `startdate/enddate` even when the range picker is empty or only
partially selected (e.g., via the Refresh button), because `dayjs(undefined)`
falls back to “now”. Guard on both elements being present before formatting
dates.
--
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]
