Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[NFC] Fix Contribution Soft Credit entity translation #19351

Merged
merged 1 commit into from
Jan 10, 2021

Conversation

mlutfy
Copy link
Member

@mlutfy mlutfy commented Jan 8, 2021

Overview

Fixes a string that is not valid English, and therefore confusing to translators.

Before / After

Entity "Contribution Softs" renamed to "Contribution Soft Credits".

ping @colemanw I think you may have added this string. Was there a length issue that required a shorter string?

Reported by @r4zoli on mattermost

@civibot
Copy link

civibot bot commented Jan 8, 2021

(Standard links)

@mlutfy
Copy link
Member Author

mlutfy commented Jan 8, 2021

Added a typo fix for "Recipientses"

@mlutfy
Copy link
Member Author

mlutfy commented Jan 8, 2021

oh no - I just realized that these are auto-generated DAOs. Need to find how to fix in XML

@mlutfy
Copy link
Member Author

mlutfy commented Jan 8, 2021

I updated the PR to fix a few XML files. Updating all xml files will take a while.

For what it's worth, I understand why CRM_Utils_String::pluralize was added, because this is a pain, but I don't think it's a good solution. This kind of magic rarely translates well.

@eileenmcnaughton eileenmcnaughton merged commit df1fc58 into civicrm:master Jan 10, 2021
@eileenmcnaughton eileenmcnaughton deleted the fixSoftLoc branch January 10, 2021 03:30
totten added a commit to totten/civicrm-core that referenced this pull request Jan 11, 2021
This is a follow-up to civicrm#19351 to
reflect recent changes in some schema files.
totten added a commit to totten/civicrm-core that referenced this pull request Jan 11, 2021
This is a follow-up to civicrm#19351 which captures the
spelling fix ("Recipientses").
@totten totten mentioned this pull request Jan 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants