yuluo-yx opened a new pull request, #4462:
URL: https://github.com/apache/rocketmq-dashboard/pull/4462

   ### Which Issue(s) This PR Fixes
   
   - Fixes #4461
   
   ### Brief Description
   
   Adds reusable message property templates to the Topic send dialog so 
operators can quickly append trace, tenant, retry audit, or order diagnostic 
properties while preserving existing input.
   
   The change includes:
   
   - A template utility that supports predefined templates, timestamp 
placeholders, form-row application, and batch `key=value` text application.
   - A Topic send dialog selector and apply action for both custom property 
entry modes.
   - Chinese and English UI translation entries for the new controls and 
messages.
   - Focused unit and page interaction tests.
   - A user-facing doc that explains the templates, placeholder expansion, and 
no-overwrite behavior.
   
   ### How Did You Test This Change?
   
   ```bash
   cd web
   npm test -- --run src/utils/messagePropertyTemplates.test.ts 
src/pages/instance/__tests__/TopicPage.test.tsx
   ```
   
   Result: 2 test files passed, 37 tests passed. Vitest printed the existing 
Ant Design/jsdom `getComputedStyle()` pseudo-element notice during the page 
tests.
   
   ```bash
   cd web
   npm run lint
   ```
   
   Result: passed with 0 errors. ESLint reported 10 pre-existing warnings in 
unrelated files: `QueueBrowser.tsx`, `dlq.tsx`, `alerts.tsx`, 
`systemAlerts.tsx`, and `LiteTopic.tsx`.
   
   ```bash
   git diff --check
   ```
   
   Result: passed.
   
   Pre-commit also ran `eslint --fix` and `prettier --write` on the staged 
TypeScript/TSX files during commit.
   
   ### Checklist
   
   - [x] One coherent change; unrelated modifications are not bundled in
   - [x] Commit subject follows Conventional Commits (`feat:` / `fix:` / 
`refactor:` / `chore:` / `docs:` / `perf:`)
   - [x] Tests added or updated for non-trivial changes, test methods named 
`...Test`
   - [x] New UI text has both Chinese and English entries under `web/src/i18n/`
   - [ ] Architecture constraints stay green (`mvn test` runs the ArchUnit 
checks; not run locally because this PR only touches frontend/docs)
   - [x] New source files carry the ASF license header
   - [x] Documentation touched where behaviour changed (README / `docs/` / 
in-app help)
   


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