maxgoedjen / secretive

Store SSH keys in the Secure Enclave
MIT License
6.99k stars 155 forks source link

Add French localization #506

Closed mtardy closed 5 months ago

mtardy commented 5 months ago

Saw your proposal to write localization in the release notes and since I love using Secretive I did the one for French 🇫🇷!

I just had hesitation on how to translate the agent_(not_)running_notice_title because it's a bit inconsistent between using "Agent" (and click to see the details) and "Secret Agent":

So I translated that to the French common name with a determinant: "L'agent"

The rest should be fine!

maxgoedjen commented 5 months ago

@mtardy this is fantastic, thank you so much for doing this! I'm gonna merge this one now, but I just noticed a handful of strings I missed (mainly around the little password/Touch ID prompts for secured keys) on my first pass. If you don't mind, can I tag you on the followup PR I'll have adding those?

agent_not_running_notice_title: Secret Agent Is Not Running
agent_running_notice_title: Agent is Running

Yeah, there's no real consistency on my part there. I should update that in English as well ;)

maxgoedjen commented 5 months ago

Those changes are here: https://github.com/maxgoedjen/secretive/pull/507 I've also consolidated both localizations (including your previous translations) into one single localization file, so it should be a little simpler to deal with in the future. Thank you again!

mtardy commented 5 months ago

@mtardy this is fantastic, thank you so much for doing this!

I'm really glad you liked it 😌! I will retake a look then since you merged https://github.com/maxgoedjen/secretive/pull/507.