qwat / QWAT

TEKSI Water module (project QWAT) - QGIS project
https://www.teksi.ch
GNU General Public License v2.0
57 stars 31 forks source link

Migration to QGIS 3 #299

Closed lbartoletti closed 4 years ago

lbartoletti commented 5 years ago

This migration consists to:

haubourg commented 5 years ago

of course +1 on my side. @ponceta @kandre Any test on your side is welcome

[funded by Charente Eaux]

lbartoletti commented 4 years ago

@ponceta @kandre Just a gentle reminder to update QGIS project to v3 :wink:

haubourg commented 4 years ago

A little Up here. Anyone willing to test the QGIS3 version of this public project?

ponceta commented 4 years ago

I made a few tests, this works fine! Thanks for the update!

image

ponceta commented 4 years ago

image

There's something strange with the use of chained filters and update but that's probably a known issue?

@dsavary Do the chain filters update correctly on QGIS 2.18.X?

kandre commented 4 years ago

Great, thanks for this update ! Regarding commits named "remove useless ui file...", do you suggest to abandon gently the ui form files to dragndrop ones in general with QGIS 3 ?

lbartoletti commented 4 years ago

Regarding commits named "remove useless ui file...", do you suggest to abandon gently the ui form files to dragndrop ones in general with QGIS 3 ?

As possible, this simplifies projects IMHO

haubourg commented 4 years ago

@kandre in fact there should be no more .ui file for forms. It's a lot simpler to maintain and QWAT is easier for newcomers. On UI is still there for the alternative geometry dialog chooser.

haubourg commented 4 years ago

There's something strange with the use of chained filters and update but that's probably a known issue?

@ponceta can you clarify what is wrong ?

kandre commented 4 years ago

As possible, this simplifies projects IMHO

@kandre in fact there should be no more .ui file for forms. It's a lot simpler to maintain and QWAT is easier for newcomers.

I do aggree, especially with the new QML widget from my point of view.

kandre commented 4 years ago

I have some new questions about this PR :

haubourg commented 4 years ago

"switch to trackable project file" : what does mean trackable project file ? TLDR, it is is of no use currently. It means we use the plugin https://github.com/opengisch/qgis_trackable_project_files/ to (try to) write the xml file in a more reproductible way so that git diff are usable. In fact, it does not work correctly yet since it works for one machine, but does not serialize the same way on another machine. I think it also requires a more recent version of QGIS, and is still a work in progress.

* "[add valve: missing checkbox for geometry alternative](https://github.com/qwat/QWAT/pull/299/commits/0d2f6846626f752b4d251f91dfae053acf3af1d5)" : I haven't found the checkbox, where is it placed ?

It should be in the dialog poping up when you start an edit session. I will raise only if you enable macros. It is the feature that allow you to choose if you want to edit both geometries or only the detailed geometry of a pipe.

* "[clean layer actions](https://github.com/qwat/QWAT/pull/299/commits/d23df68782babbb290ed88fd22769a6cae90a746)": if one clean those actions, does the postgresql audit used for data changes history still working ?

Yes, those were automatically added by the old plugin Pg-audit_91 but are of no use currently, all is done on the DB side.

ponceta commented 4 years ago

@ponceta can you clarify what is wrong ?

I've opened an issue here, since this is QGIS related.

https://git.oslandia.net/Support-projects/helpdesk_qwat_qgep/issues/86

lbartoletti commented 4 years ago

@ponceta Can we merge it?

ponceta commented 4 years ago

Yes! Please!

haubourg commented 4 years ago

@lbartoletti I let you the honor of pressing the green button :)

lbartoletti commented 4 years ago

@lbartoletti I let you the honor of pressing the green button :)

:boom::boom::boom:

lbartoletti commented 4 years ago

Done :)

Thanks

ponceta commented 4 years ago

QWAT is now officially QGIS 3.X Ready! Would be a nice thing to put on the qwat news! I'll check that with Amandine.