Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Modularization for beta v2 #220

Merged
merged 50 commits into from Sep 2, 2019
Merged

Modularization for beta v2 #220

merged 50 commits into from Sep 2, 2019

Conversation

SimonSAMPERE
Copy link
Collaborator

see #180

SimonSAMPERE and others added 30 commits July 9, 2019 15:33
PyQt Checkable Combobox for keywords selection
#178
layers can be added from files or services (OGC and ESRI)
#176

The most of the quicksearch managment is out of isogeo.py
Modularization 1 - add layers, metadata synchronization, make quick searches
#179
splitting old modules/results.py into new modules modules/results/cache.py and modules/results/display.py
using modules/results/cache.py into modules/results/display.py and modules/results/display.py  into isogeo.py
Modularization 2 - manage layer adding cache
#177

removing old `api.py` and adapting `__init__.py` files of `modules` folder
…ration

#177

replacing old IsogeoPlgApiMngr with new Authenticator or ApiRequester
introducing ApiRequester in Isogeo by signal and slot system
+ cleaning Isogeo
#177

adapting traduction to the renaming of the old IsogeoPlgApiMngr
Modularization 3 - interaction with the API
#215
#177
Complete reorganization of the code to allow to send shares request to Isogeo's API asynchronously
#177 #215
Adapting Isogeo main class to new signals and slots system of ApiRequester
Guts and others added 19 commits August 23, 2019 14:32
Modularization 4 - Displaying shares informations
#210
- adding the get_tags method remove from auth.py module
- correcting the get_tags method to match the fields names with the parameters names
- ajusting search_sig and details_sig signals to make them pass tags as dicts to their slot
#210
adapt it to the adjustment of the requester.py's signals
#214
#210

into quicksearch managment and results display
#214
#210
- docstring
- intergate SearchFormManager use in the script
- major change in the way search work to make the search form modularization easier and simplify operation
- remove methods that are no longer useful or that now belong to the search_form.py module
Remove the management of the search form from the main class **Isogeo** (`isogeo.py`)
see #217 for more details about code changes and improvements
@SimonSAMPERE SimonSAMPERE added enhancement Amélioration du fonctionnement existant code art Qualité du code source PyQGIS labels Sep 2, 2019
@SimonSAMPERE SimonSAMPERE self-assigned this Sep 2, 2019
@SimonSAMPERE SimonSAMPERE added this to In progress in Modularisation via automation Sep 2, 2019
Modularisation automation moved this from In progress to Reviewer approved Sep 2, 2019
@SimonSAMPERE SimonSAMPERE merged commit 17bd2ee into master Sep 2, 2019
Modularisation automation moved this from Reviewer approved to Done Sep 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code art Qualité du code source enhancement Amélioration du fonctionnement existant
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants