MerginMaps / mobile

Mobile app for Mergin Maps 📲. Open QGIS projects and manage vector data on your phone, even offline 📶 (Android/iOS/Windows app)
https://merginmaps.com
GNU General Public License v2.0
280 stars 62 forks source link

Fix typo: replace treshold with threshold #3492

Closed volcan01010 closed 3 weeks ago

volcan01010 commented 4 weeks ago

Summary

This merge request fixes a typo in the interface, correcting the spelling of "threshold".

To check

The file diff shows that changes that were made. There are no changes to functionality.

I don't have a Qt development environment, so I wasn't able to compile the binary language files. This will need to be checked before the code is merged. I apologise for submitting a half-finished pull request.

$ git grep treshold
Binary file app/i18n/input_az.qm matches
Binary file app/i18n/input_az_AZ.qm matches
Binary file app/i18n/input_ca.qm matches
Binary file app/i18n/input_de.qm matches
Binary file app/i18n/input_es.qm matches
Binary file app/i18n/input_es_419.qm matches
Binary file app/i18n/input_es_ES.qm matches
Binary file app/i18n/input_et.qm matches
Binary file app/i18n/input_et_EE.qm matches
Binary file app/i18n/input_fr.qm matches
Binary file app/i18n/input_fr_FR.qm matches
Binary file app/i18n/input_hr.qm matches
Binary file app/i18n/input_hr_BA.qm matches
Binary file app/i18n/input_hr_HR.qm matches
Binary file app/i18n/input_hu.qm matches
Binary file app/i18n/input_it.qm matches
Binary file app/i18n/input_pl.qm matches
Binary file app/i18n/input_pt_BR.qm matches
Binary file app/i18n/input_pt_PT.qm matches
Binary file app/i18n/input_sk.qm matches
Binary file app/i18n/input_sk_SK.qm matches

Closes https://github.com/MerginMaps/mobile/issues/3493

PeterPetrik commented 3 weeks ago

Thanks for fix! If you struggle to setup devel environment, ping us on merginmaps.com/community/join

volcan01010 commented 3 weeks ago

I managed to update the .qm files by doing the following:

sudo apt install qttools5-dev-tools
bash scripts/release_i18n_qm.bash

I think that the merge request is ready now.

PeterPetrik commented 3 weeks ago

generally it would be enough to update .qml file - we update all the other files before release in one go