RockteMQ-AI commented on issue #4842: URL: https://github.com/apache/rocketmq-dashboard/issues/4842#issuecomment-5770633650
**Issue Evaluation** Category: `bug` | Status: **Confirmed** This is a valid pagination defect. The `onChange` handler in `notificationDeliveries.tsx` stores the page value from antd even when only the page size changes, stranding the table on a nonexistent page when the new size reduces the total page count below the current page. **Root Cause:** The pagination `onChange` does not reset the page when `nextPageSize !== pageSize`, unlike the audit page (`audit.tsx`) which already handles this case correctly. **Impact:** Notification Deliveries page becomes unusable (empty table) after changing page size from a high page number. **Severity:** Medium — UX defect with a clear fix pattern already established in the codebase. The referenced PR #4833 appears to address this correctly by adopting the audit page's reset-on-size-change pattern. --- *Automated evaluation by RockteMQ-AI* -- 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]
