[x] NRU information panel, some fields are not populated even if related values are available in the output of the backed if the user has proper permissions:
Calculated area (m²): surfc field available in the first getFIC request output
Owner account: app_nom_usage field in the 2nd getFIC request output
Owner (s): it can be an array, use the comptecommunal field in the 2nd getFIC request output
[x] ADS information panel: use the string Education managing sector instead of Education sector
[x] Provide a translation files for French using the same translation strings of the current plugin for mapfishapp: ensure they are properly working by changing the browser default language.
[x] At the moment doesn't seems possible to configure the plugin because all configuration properties are hardcoded: we have to externalize all relevant config properties of the plugin like in the old version, where the following files are available:
The aim is to allow the context admin to configure them in the application context wizard in the same way of all the other plugins. The default properties values, if possible, must be reported as they are as a default configuration in the app context wizard, ready to be changed by the admin if needed. Now it is empty and the admin doesn't know which the default configuration is and, since config properties are hardcoded, it is not possible to configure them in the wizard
[x] Document the plugin configuration properties available for the administrator in the context wizard:
We can update the readme file in the extension repo for this putting also some sample (eg. the default config)
The doc link in the app context wizard for the plugin should point to the readme
[x] NRU information panel, some fields are not populated even if related values are available in the output of the backed if the user has proper permissions:
[x] ADS information panel: use the string Education managing sector instead of Education sector
[x] Provide a translation files for French using the same translation strings of the current plugin for mapfishapp: ensure they are properly working by changing the browser default language.
[x] At the moment doesn't seems possible to configure the plugin because all configuration properties are hardcoded: we have to externalize all relevant config properties of the plugin like in the old version, where the following files are available:
Relevant proeprties for this are for example: https://github.com/sigrennesmetropole/addon_urbanisme/blob/master/src/addon/urbanisme/config.json#L8 https://github.com/sigrennesmetropole/addon_urbanisme/blob/master/src/addon/urbanisme/config.json#L9 https://github.com/sigrennesmetropole/addon_urbanisme/blob/master/src/addon/urbanisme/config.json#L16
Relevant proeprties for this are for example: https://github.com/sigrennesmetropole/addon_urbanisme/blob/master/src/addon/urbanisme/manifest.json#L10 https://github.com/sigrennesmetropole/addon_urbanisme/blob/master/src/addon/urbanisme/manifest.json#L11
The aim is to allow the context admin to configure them in the application context wizard in the same way of all the other plugins. The default properties values, if possible, must be reported as they are as a default configuration in the app context wizard, ready to be changed by the admin if needed. Now it is empty and the admin doesn't know which the default configuration is and, since config properties are hardcoded, it is not possible to configure them in the wizard