PnX-SI / gn_mobile_occtax

Application mobile pour la saisie dans le module Occtax de GeoNature
GNU General Public License v3.0
13 stars 5 forks source link

[1.3.0] Saisie dénombrement : pas de min et max #96

Closed Splendens closed 5 months ago

Splendens commented 3 years ago

Bonjour,

Je teste la saisie mobile avec les 2 appli sync et occtax en 1.3.0, et GeoNature 2.7.5.

J'ai suivi la doc pour l'installation et je peux bien saisir sur mobile. Par contre, le min et le max des dénombrements sont à 1, et je n'ai pas de champ pour changer ces valeurs. De mémoire, avec la version 1.2 d'occtax_mobile, les champs min et max apparaissaient bien.

Est-ce un soucis avec le settings.json par défaut ?

Merci d'avance pour votre aide !

DonovanMaillard commented 3 years ago

Bonjour,

En effet la conf permet de faire apparaître ces champs. En principe ca devrait être le cas par défaut... je vérifierai a l'occasion. Dans tous les cas ces champs peuvent être ajoutes dans la conf.

DonovanMaillard commented 2 years ago

Bonjour @Splendens , est ce que tu as pu solutionner le soucis ?

Splendens commented 2 years ago

Salut ! Oui, en rajoutant les champs de min et max dans la conf, merci ! Et effectivement, je suis d'accord avec toi, ces champs devraient être présents par défaut.

camillemonchicourt commented 2 years ago

C'est une erreur de conf par défaut. Ils devraient être affichés par défaut comme c'était le cas initialement. A remettre affiché par défaut dans une prochaine version.

DonovanMaillard commented 1 year ago

@sgrimault sera-t-il possible de corriger la valeur par défaut pour ces deux champs count_min et count_max pour les prochaines releases ? La doc indique que les champs count_min et count_max sont affichés par défaut, mais dans la pratique s'ils ne sont pas explicitement activés dans le settings.ini, ils ne sont pas affichés :)

sgrimault commented 1 year ago

L'idée est que l'on considère que ces champs restent visibles et obligatoires quelque soit la configuration ?

DonovanMaillard commented 1 year ago

Non non du tout. C'est que si rien n'est renseigné dans le settings.ini, ces champs soient affichés. Actuellement il faut impérativement ajouter

{ "key": "count_min", "visible": true },

Si on met clairement visible:false, alors là seulement le champs ne s'affiche pas.

Dans le readme il est indiqué que le champs min devrait etre affiché par défaut mais ce n'est pas le cas de mon coté quand je teste.

camillemonchicourt commented 5 months ago

Les champs de dénombrement sont bien présent dans l'interface par défaut, même si cela n'est pas indiqué explicitement dans la config.