dheeraj12347 commented on PR #12681:
URL: https://github.com/apache/cloudstack/pull/12681#issuecomment-3943174516
@shwstppr You were absolutely correct! The first commit (2244c01) only had
formatting changes. I've now pushed a new commit (cf2f331) with the actual 404
bug fix:
1. **Core fix (Line 481)**: Changed `this.$router.go(-1)` →
`this.$router.push({ path: '/template' })` in handleCancel. This explicitly
redirects to Templates list when all zones are deleted, avoiding the 404.
2. **v-html formatting (Lines 220-222)**: Reverted to template literal style
to match UI conventions
3. **pageSizeOptions**: Confirmed inline array format
All issues from Copilot's review have been addressed. Ready for re-review!
--
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]
