zmuxuny opened a new pull request, #7180:
URL: https://github.com/apache/rocketmq-dashboard/pull/7180

   ### Which Issue(s) This PR Fixes
   
   Fixes #7179.
   
   ### Brief Description
   
   Add an in-place Retry action to the resource operation timeline error. A 
failed page read currently removes the pagination controls; recovering requires 
changing dates or closing/reopening the modal, which loses the current view.
   
   Reuse common.retry and include a reload generation in the local response 
identity. Retrying immediately replaces the old error with loading while 
retaining the same resource, cluster scope, dates, page and page size. The 
existing request-version guard still ignores obsolete responses. No backend/API 
changes or new UI strings.
   
   ### How Did You Test This Change?
   
   - Five new real-AntD regressions: English/Chinese initial-read recovery; 
filtered page 2 with clusterIdMissing; repeated failure followed by a 
successful empty result; and a late retry after switching resources.
   - Exact final tests against unchanged production: 5 failures, each at the 
absent Retry action.
   - Final related suite: 61 tests passed across 5 files.
   - Forced TypeScript build, changed-file ESLint/Prettier and git diff --check 
passed. Independent review of the final diff found no blocking issue.
   
   Commands from web (NODE_OPTIONS=--max-old-space-size=768):
   ```text
   node node_modules/vitest/vitest.mjs run 
src/pages/ops/__tests__/ResourceOperationTimelineModal.test.tsx 
src/pages/ops/__tests__/AuditPage.test.tsx 
src/pages/ops/__tests__/auditPresentation.test.ts src/api/audit.test.ts 
src/services/opsService.test.ts --maxWorkers=1
   node node_modules/typescript/bin/tsc -b --force
   node node_modules/eslint/bin/eslint.js 
src/pages/ops/ResourceOperationTimelineModal.tsx 
src/pages/ops/__tests__/ResourceOperationTimelineModal.test.tsx
   node node_modules/prettier/bin/prettier.cjs --check 
src/pages/ops/ResourceOperationTimelineModal.tsx 
src/pages/ops/__tests__/ResourceOperationTimelineModal.test.tsx
   ```
   
   Only service responses are controlled; the modal, date controls and 
pagination are real components. No full frontend suite, bundle build, backend 
tests or live cluster were exercised.
   
   ### Checklist
   
   - [x] One coherent recovery change; Conventional Commit title
   - [x] Regression tests cover the actual modal and repeated/obsolete requests
   - [x] Existing bilingual Retry wording reused
   - [x] No new source files, dependencies or public API changes
   


-- 
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]

Reply via email to