Skip to content

Conversation

@niko-exo
Copy link

@niko-exo niko-exo commented Jan 23, 2026

ref: https://app.clickup.com/t/86b87faud

86b87faud - feat: modify caption when deleting custom field value on list

Changelog

  • Replace string on i18n file.
  • Delete argument on translate call.

Links

86b87faud - Modify text when deleting field value

Evidence

2026-01-23_17-17-56.mp4

Summary by CodeRabbit

  • Bug Fixes
    • Updated the delete confirmation message for meta field values to provide clearer, more explicit instruction, improving user understanding before confirming deletion.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 23, 2026

📝 Walkthrough

Walkthrough

Removed dynamic interpolation of a meta field value name in a delete confirmation dialog and updated the corresponding English translation string to use a more explicit instruction message instead of a generic confirmation prompt.

Changes

Cohort / File(s) Summary
Delete Confirmation Message Updates
src/components/mui/formik-inputs/additional-input/meta-field-values.js, src/i18n/en.json
Removed dynamic metaFieldValue.name interpolation from delete confirmation text; updated English translation from generic "Are you sure you want to delete meta field value" to explicit "Please verify you want to delete the added value"

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested reviewers

  • smarcet

Poem

🐰 A message refined, no names to display,
Just verify once before you delete the day,
Two files aligned in harmony's dance,
Confirmation confirmed at a glance!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: modifying the deletion confirmation caption for custom field values in a list, which is reflected in both the i18n update and the code change.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

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

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.

4 participants