Cadasta / cadasta-qgis-plugin

[DEPRECATED] QGIS plugin to create, edit, upload and download Cadasta projects
GNU General Public License v3.0
4 stars 7 forks source link

added helper #66

Closed meomancer closed 7 years ago

meomancer commented 7 years ago

Added helper fix #45 :

timlinux commented 7 years ago

For a simple help system you may want to look at the messaging classes from InaSAFE (@ismailsunni we could hive those out to a separate package / repo). They have the advantage that you do not need separate workflows for translating project documentation and UI stuff. See e.g.

https://github.com/inasafe/inasafe/blob/develop/safe/gui/tools/help/function_options_help.py

ismailsunni commented 7 years ago

Hi @timlinux Yeah, I think we can do it. But the messaging classes are not completely separated from the other code. It imports tr, get_unicode, and resource_path. Perhaps we can move them out to a separate package. I checked, those three helper methods are independent from the rest of the code. Then we can have qgis_plugin_utilities repository that people can re-use it also.

meomancer commented 7 years ago

screenshot from 2017-01-03 13 12 53

Using message class

ismailsunni commented 7 years ago

Looks very nice to me

Gustry commented 7 years ago

Then we can have qgis_plugin_utilities repository that people can re-use it also.

They are many files that we need to copy and paste when we work on another plugin : messaging system, datastore (folder, geopackage, postgis ...), helpers to load a ressource, helpers to load a vector/raster layer for tests, debug helpers ... It could be very nice if we can refactor these.

timlinux commented 7 years ago

Looking nice @meomancer !

ismailsunni commented 7 years ago

They are many files that we need to copy and paste when we work on another plugin : messaging system, datastore (folder, geopackage, postgis ...), helpers to load a ressource, helpers to load a vector/raster layer for tests, debug helpers ... It could be very nice if we can refactor these.

Yeah, totally agree with this.

ismailsunni commented 7 years ago

Hi @meomancer if you want to merge this PR, please go. The PR is getting bigger. You can make another PR for the Update Project Helper.