synapse-admin icon indicating copy to clipboard operation
synapse-admin copied to clipboard

Remove not needed `translate`

Open dklimpel opened this issue 3 years ago • 3 comments

dklimpel avatar Feb 03 '22 16:02 dklimpel

Seems we have to figure out when translate() is really needed. I found some places where it is not needed but others (e.g. inside <Button>) where it is needed. Do you have some documentation about that?

awesome-michael avatar Mar 22 '22 17:03 awesome-michael

Sorry, I have no docs for it. There are also changes, e.g: https://github.com/marmelab/react-admin/blob/next/UPGRADE.md#menuitemlink-automatically-translates-primarytext

dklimpel avatar Mar 22 '22 17:03 dklimpel

This is the official doc: https://marmelab.com/react-admin/Translation.html#usetranslate-hook

dklimpel avatar Mar 23 '22 06:03 dklimpel