dheeraj12347 commented on PR #12681:
URL: https://github.com/apache/cloudstack/pull/12681#issuecomment-3943172943
@shwstppr Thanks for your question! You were correct—the first commit
(2244c01) only had formatting changes. I've now pushed a new commit (cf2f331)
that includes the actual 404 bug fix:
**What was added in the new commit:**
1. **The core navigation fix** (Line 481): Changed `this.$router.go(-1)` to
`this.$router.push({ path: '/template' })` in the `handleCancel` method. This
ensures that when all zones are deleted after bulk delete, users are explicitly
redirected to the Templates list instead of navigating back in browser history
(which could lead to a 404).
2. **v-html formatting** (Lines 220-222): Reverted to template literal style
`"`<b>${selectedRowKeys.length} ...`"` to match existing UI conventions.
3. **pageSizeOptions formatting**: Confirmed inline array format is used.
The previous comments from Copilot about missing the actual bug fix have now
been addressed. Please let me know if you'd like me to make any other
adjustments!
--
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]
