Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Rename customer "Contact" field to "Contact person" - #6079

Open
JoeJoeflyn wants to merge 1 commit into
kimai:mainfrom
JoeJoeflyn:fix/customer_contact_wording
Open

Rename customer "Contact" field to "Contact person"#6079
JoeJoeflyn wants to merge 1 commit into
kimai:mainfrom
JoeJoeflyn:fix/customer_contact_wording

Conversation

@JoeJoeflyn

@JoeJoeflyn JoeJoeflyn commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

Fixes #5770.

Description

The customer contact field stores a person name, but the label "Contact" was ambiguous and could be read as a contact method (email/phone). The same translation key was shared with the InvoiceTemplate contact field, which is a free-text textarea with different semantics.

Adds a dedicated contact_person translation key for the customer field (form label, detail view, list column header, export column header) while keeping the original contact key for InvoiceTemplate, per the maintainer's plan in #5770.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • I verified that my code applies to the guidelines (composer code-check)
  • I updated the documentation (see here)
  • I agree that this code is used in Kimai (see license)
screenshot-2026-07-26_21-50-08 screenshot-2026-07-26_21-50-22 screenshot-2026-07-26_21-50-33 screenshot-2026-07-26_21-50-51

@JoeJoeflyn
JoeJoeflyn force-pushed the fix/customer_contact_wording branch from 33dfc66 to 9f7db24 Compare August 2, 2026 19:38
@codecov

codecov Bot commented Aug 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.81%. Comparing base (8f0514a) to head (ec4a0ab).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #6079      +/-   ##
============================================
+ Coverage     88.75%   88.81%   +0.05%     
- Complexity    10090    10093       +3     
============================================
  Files           889      889              
  Lines         32152    32163      +11     
============================================
+ Hits          28538    28564      +26     
+ Misses         3614     3599      -15     
Files with missing lines Coverage Δ
src/Controller/CustomerController.php 94.50% <100.00%> (ø)
src/Entity/Customer.php 100.00% <ø> (ø)
src/Form/CustomerEditForm.php 100.00% <100.00%> (ø)
src/Form/InvoiceTemplateForm.php 100.00% <100.00%> (ø)

... and 11 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@kevinpapst kevinpapst left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you 👍

I would like to see another approach:
use the bin/console kimai:translations --copy-resname command to copy the contact key to a new contact_info key.

Change the english contact key to Contact person and the de one to Kontaktperson.

Then use the new contact_info key for the InvoiceTemplate label.

That prevents us from having missing translations in both screens and we do not need to remap the Customer key to a different label.

Copy the contact translation key to contact_info for the InvoiceTemplate
label, and change the customer contact label to "Contact person" (en) /
"Kontaktperson" (de).
@JoeJoeflyn
JoeJoeflyn force-pushed the fix/customer_contact_wording branch from 9f7db24 to ec4a0ab Compare August 5, 2026 18:53
@JoeJoeflyn

Copy link
Copy Markdown
Contributor Author

Thanks, fixed.

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.

Improve customer "Contact" wording

2 participants