Skip to content

fix: adjust add on type delete message#1016

Open
tomrndom wants to merge 1 commit into
masterfrom
fix/add-on-type-delete-msg
Open

fix: adjust add on type delete message#1016
tomrndom wants to merge 1 commit into
masterfrom
fix/add-on-type-delete-msg

Conversation

@tomrndom

@tomrndom tomrndom commented Jul 21, 2026

Copy link
Copy Markdown

ref: https://app.clickup.com/t/9014802374/86bb0t09d

Signed-off-by: Tomás Castillo tcastilloboireau@gmail.com

Summary by CodeRabbit

  • User Interface
    • Updated the English confirmation message shown when deleting an Add-On Type for clearer wording.

Signed-off-by: Tomás Castillo <tcastilloboireau@gmail.com>
@coderabbitai

coderabbitai Bot commented Jul 21, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 99a48f94-6109-406b-b7cc-e7adae8eb96f

📥 Commits

Reviewing files that changed from the base of the PR and between b4ef1d3 and fd2294c.

📒 Files selected for processing (1)
  • src/i18n/en.json

📝 Walkthrough

Walkthrough

The English Add-On Type deletion confirmation prompt is reworded from “Are you sure you want to delete …” to “Please verify you want to delete …”, while retaining the {name} interpolation.

Changes

Add-On Type deletion prompt

Layer / File(s) Summary
Update deletion confirmation text
src/i18n/en.json
The delete_add_on_type_warning translation wording is updated while preserving the {name} variable.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: smarcet, santipalenque

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the change: it adjusts the Add-On Type delete warning message.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/add-on-type-delete-msg

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant