Nitrokey / nitrokey-app

Nitrokey's Application (Win, Linux, Mac)
https://www.nitrokey.com/
288 stars 55 forks source link

Firmware export: tell user where the export was saved #363

Closed alex-nitrokey closed 3 years ago

alex-nitrokey commented 6 years ago

Expected behaviour

Tell user where the export was saved, when firmware export to file function is used.

Current behaviour

Only success of export is mentioned

A user requested the feature (point 1) to being able to decide where the export is saved. As far as I can see this is done by the firmware and therefore it is not desirable to change something on this behaviour. Instead, the user should at least get informed to avoid confusion.

alex-nitrokey commented 6 years ago

Sorry for the duplication, I had a bit trouble managing git :blush:

Is this how it works? How will the other languages notice the change? Or how is the workflow working? (sorry for the questions...)

szszszsz commented 6 years ago

Hi Alex!

No problem! Please make the PR with a information, which issue will be fixed with it. Details should be here or similar sites explaining Github workflow.

Regarding the translation - how about adding the exported file name as well? I think this is firmware.bin, but I am not sure.

alex-nitrokey commented 6 years ago

No problem! Please make the PR with a information, which issue will be fixed with it. Details should be here or similar sites explaining Github workflow.

Yeah, I already realized that making a PR with master branch is a bad idea, but somehow forgot the other day. This messed up some things.

Good idea, I will add it in the PR I'll create. I just wasn't sure if that's the way to change the original string...

szszszsz commented 3 years ago

Fixed through https://github.com/Nitrokey/nitrokey-app/pull/364, but missed automatic closing.