Produits Entr'ouvert » Welco

return data['id'], data.get('url_backoffice')

Mais dans #23939 c'est backoffice_url finalement.

(pas lu 0004)

Un mot sur la manière dont on pourrait tester 0004 ?

Pour 0004 à part se dire qu'à partir du moment (...)

Le 0004 c'est le code par rapport à Maarch; passer par settings.json etc. pour le contrib.alfortville ne me pose aucun problème.

• réception des mails par la commande de management
• qualification puis envoie dans w.c.s.
• réception et stockage de backoffice_url
• notification de Maarch du changement de statut du courier, du numéro de la demande dans w.c.s. et de l'URL BO du formulaire

Je reconnais que c'est fait via mock plutôt qu'en instanciant w.c.s, est-ce cela qui te manque ?

Non, je parle d'exécuter le code, en vrai, pas de tests unitaires.

On a une instance Maarch de test, j'ai testé avec aussi mais en live, tu voudrais une instance Maarch de test chez nous ? Dès que c'est acké je comptais brancher la démo de dév dessus.

Des indications quelconque sur la manière dont prendre le code et l'exécuter en local et avoir des données dedans etc. le paramétrage à mettre, peu importe pour moi qu'il pointe une instance chez nous ou ailleurs, ou qu'il faille installer un truc en local, etc.

Ah oui, il suffit d'ajouter cette configuration dans settings.json:

MAARCH_FEED = {
    'ENABLE': True,
    'URL': '...',
    'USERNAME': '...',
    'PASSWORD': '...',
}

Et c'est tout.

Quand la configuration (MAARCH_FEED) est absente/invalide, get_maarch() ne produit pas d'erreur:

>>> from welco.sources.mail.utils import get_maarch
>>> maarch = get_maarch()
>>> print(maarch)
None

Avec la bonne config, j'ai tenté:

>>> from welco.sources.mail.utils import get_maarch
>>> maarch = get_maarch()
>>> c = maarch.new_courier_with_file(open('mon.pdf').read(), format='pdf', status='GRC', type_id='101')
Traceback (most recent call last):
  File "<console>", line 1, in <module>
AttributeError: 'WelcoMaarchCourrier' object has no attribute 'new_courier_with_file'

new_courrier_with_file() avec deux 'r'.

Poussé sur une branche http://git.entrouvert.org/welco.git/log/?h=wip/22550-interface-avec-Maarch

new_courrier_with_file() avec deux 'r'.

ah oui; par la suite:

>>> c = maarch.new_courrier_with_file(open('mon.pdf').read(), format='pdf', status='GRC', type_id='101')
>>> maarch.post_courrier(c)
Traceback (most recent call last):
  File "<console>", line 1, in <module>
  File "/home/chris/src/welco/welco/sources/mail/maarch.py", line 227, in post_courrier
    response = self.post_json(self.post_courrier_url, courrier.post_serialize())
  File "/home/chris/src/welco/welco/sources/mail/maarch.py", line 85, in post_serialize
    payload['fileFormat'] = self.fileFormat
AttributeError: 'Courrier' object has no attribute 'fileFormat'

Remplace format='PDF' par fileFormat='PDF' l'API a un peu bougé mais ça ne fait pas vraiment partie de la cible voulu (à aucun moment on est amené à poster des fichiers dans Maarch depuis Welco, ça facilite juste les tests ici).

qualif: store keep formdata backoffice URL (#22550)

Ack

settings: remove alfortville default validation steps (#22550)

A mon avis il faut laisser les settings, parce qu'utilisés dans views.py, et
donc juste les vider, avec des ptits exemples et zou, genre:

# Add validation circuit on specific sources, example:
# VALIDATION_STEPS = {'mail': ['done-qualif', 'done-dgs', 'done-dga']}
# VALIDATION_STEP_LABELS = {
#     'done-qualif': _('Send to DGS'),
#     'done-dgs': _('Send to DGA'),
#     'done-dga': _('Send to service')}

VALIDATION_STEPS = {}
VALIDATION_STEP_LABELS = {}

mail: feed from MaarchCourrier (#22550)

Faudrait pas utiliser "Mail.registered_mail_number" car ça s'affiche comme "numéro du recommandé", et c'est peut-être une feature pertinente dans un système de gestion de courrier (pas vraiment spécifique Alfortville, et donc plutôt à laisser comme tel). Plutôt créer un "Mail.external_mail_id" ?

J'aurais bien vu welco/sources/mail/__init__.py::AppConfig.association_post_save dans welco/sources/mail/maarch.py. Et j'y ajouterais un if not maarch: return juste après le maarch = get_maarch()

Allez, aussi, des "(C) 2015" qui trainent

Voilà en relecture de base, je teste quand même "grandeur nature" ce soir ; Christophe si ça t'amuse tu peux continuer aussi, mais uniquement si ça t'amuse.

Ok avec :

-            payload['fileFormat'] = self.fileFormat
+            payload['fileFormat'] = self.format

>>> maarch.post_courrier(c)
<Courrier pk:111 status:GRC>
>>> print c.pk
111
>>> maarch.get_mail(110)
<Courrier pk:110 status:GRC>
>>> maarch.get_mail(112)
Traceback (most recent call last):
  File "<console>", line 1, in <module>
  File "/home/chris/src/welco/welco/sources/mail/utils.py", line 41, in get_mail
    return self.get_courriers(clause="res_id=%s" % mail_id)[0]
IndexError: list index out of range
>>> maarch.get_mails()
[<Courrier pk:110 status:GRC>, <Courrier pk:111 status:GRC>]

Voilà j'ai poussé sur la branche avec vos commentaires traités dans des commits distincts.

Pour le lancement en cron de feed_mail_maarch, avec un tenant_command --all-tenants, il faudrait ajouter ça je pense :

--- a/welco/sources/mail/management/commands/feed_mail_maarch.py
+++ b/welco/sources/mail/management/commands/feed_mail_maarch.py
@@ -38,6 +38,8 @@ class Command(BaseCommand):
     def handle(self, *args, **kwargs):
         verbosity = kwargs['verbosity']
         maarch = get_maarch()
+        if not maarch:
+            if verbosity > 1:
+                self.stdout.write('Maarch not configured')
+            return

         maarch_mails = maarch.get_mails()
         count = 0

(et je peine encore à tester "en vrai", c'est en cours)

(note : intégrer ce que je propose dans la note 40 stp, ie ne pas faire de feed sur les tenants où get_maarch==None)

Je vais intégrer ta remarque. Par contre j'intègre déjà le external_id dans le template, dans un champ data- au cas où.

Ça a été poussé mais ça a cassé jenkins.

J'ai tout passé en tox et j'ai ajouté l'installation de pylint dans le bout de shell dans la config du job jenkins, c'est pas idéal mais c'est vrai que nos commandes pylint référence un fichier local à la bécane jenkins donc bon...