use 'domain' argument in ts()
Hi @artfulrobot . This is the only issue I've found in the extension review so far. It's not a requirement, but I recommend addressing it, as it can help translators to give non-English speakers a better experience.
Extension developers are asked to pass the second 'domain' argument in ts()
function calls, to help translators distinguish between core terms and extension-specific terms. Reference: https://docs.civicrm.org/dev/en/latest/translation/extensions/#for-developers-correct-usage-of-the-ets-function Extensions built with newer versions of civix can use the E::ts()
method to save some keystrokes and avoid typing that second parameter every time.