-
Notifications
You must be signed in to change notification settings - Fork 62
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
Custom field translation #950
Custom field translation #950
Conversation
# Conflicts: # template-contacts-new.php
I am not sure why the build failed, If anyone can figure out what the error in TravisCI is that would be a huge help. |
Thanks @micahmills |
I'm also not sure why the travis version of the script.min.js is different than the one we are generating. I'll look at it more later |
replacing the translation when the object is created is probably a better option. I am not actually sure where/when that object is created though. |
Have a look at the |
This reverts commit e9a3402.
I am not sure why the minified script files are failing in Travis. I have merged master in and run gulp. Everything seems to be up to date, but Travis doesn't like it for some reason. |
@mikeallbutt found a bug with this PR as it stands. If the DT instance is in Language A, but the user has his language set to Language B. If he goes into the Admin and adds a translation for Language B it will appear in the default field as well. This is because we are using the "dt_custom_fields_settings_after_combine" hook in dt-contacts/contacts-post-type.php:753 to update the object label with the translated version when we create the Object, that then populates the default text input instead of the the actual default language. If the user then saves the input in this state it will make the translation the default label instead of the original. |
I think a pull from master will resolve the phpcs issue. |
…tover console.log in JS
…change the language of the label being shown. This will work for now, but will have to be changed if we ever fully translate the Admin Dashboard.
so far this makes the Custom Fields, and Custom Tiles translatable in the Admin UI. I just need to add the Custom Lists. |
This bug should be fixed as well. You can now change the default label if your user isn't using the default language. |
Brilliant @micahmills ! |
…he code that added it in the various custom fields.
…n functions there.
…aused field_key translations not to show up in the UI
@micahmills i look forward to using this feature! thank you! |
Added Translation fields to Custom Fields so you can add a custom translation to an existing key, or add a translation to your custom fields.
I also updated all (or I got them all) of the usages of the label key on the front-end.