-
-
Notifications
You must be signed in to change notification settings - Fork 857
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
Spanish translation for Base Partner Merge for branch 7.0 and a fix #128
Conversation
Hey @cesarcruz, thank you for your Pull Request. It looks like some users haven't signed our Contributor License Agreement, yet.
Appreciation of efforts, |
Translations will soon be managed on Transifex: Feel free to join the OCA Translators team and translate there. |
Ok, @max3903 thanks |
@cesarcruz would it be possible to split the missed condition to a separate PR? I am not qualified to check the Spanish translation, but I would like to validate/merge the condition. |
I'm not sure how to split PR, I created a PR for spanish translation, then added new commit with fix condition and it automatically was added to the PR, I will check how to do that, thanks |
@cesarcruz what you want to do is move this commit to another branch, use this other branch for the new PR, and rebase this branch so the commit is not included. |
@cesarcruz To split your commits, you need to do them on separate branches. Best practice is to create always a new branch for a new fix/feature. You can create a new branch on top of last commit of partner-contact and then cherry pick the commit you want:
(you will have to work with remotes Otherwise, you can create a new branch and going on it by doing |
I set it as need fixing while you are splitting the 2 fixes |
I close this PR as it seems abandoned. |
Added
i18n/base_partner_merge.pot
andi18n/es.po
files in modulebase_partner_merge
with all strings translated to spanish.UPDATE
Added new commit with a fix in base_partner_merge.
Problem: When an admin user try to merge contacts with different emails got a wrong message:
Since the logged in user is an admin it shouldn't display this error.
In branch 8.0 there is a condition asking for superuser, but in branch 7.0 don't, so I just added the condition to the branch 7.0 and now it works. The condition has been removed from this file in commit e0a359f