Project

General

Profile

Development #50087

base adresse, passer par un job pour la mise à jour initiale des rues

Added by Frédéric Péters 14 days ago. Updated 7 days ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
12 Jan 2021
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Pour le moment on a :

    def hourly(self):
        super(BaseAdresse, self).hourly()
        self.clean_addresses_cache()
        # don't wait for daily job to grab data
        if self.get_zipcodes() and not self.get_streets_queryset().exists():
            self.update_streets_data()
        if not CityModel.objects.exists():
            self.update_api_geo_data()

mais on pourrait ne pas attendre une heure, avoir un job créé quand le connecteur est créé (pour la partie CityModel) et quand son attribut zipcode est modifié (pour la partie StreetModel).


Files

Associated revisions

Revision d4c814f6 (diff)
Added by Valentin Deniaud 14 days ago

base_adresse: use job to fetch data at create and edit (#50087)

Revision beddcdc7 (diff)
Added by Valentin Deniaud 14 days ago

Revert "base_adresse: use job to fetch data at create and edit (#50087)"

This reverts commit d4c814f6a7e281d52a8680576b5b94c3fff2fd52.

Revision 82b9459c (diff)
Added by Valentin Deniaud 7 days ago

base_adresse: use job to fetch data at create and edit (#50087)

History

#1

Updated by Valentin Deniaud 14 days ago

  • Assignee set to Valentin Deniaud
#2

Updated by Valentin Deniaud 14 days ago

Aïe, gros fail de git-redmine (ma faute), j'ai envoyé dans master au lieu de soumettre le patch :'(

#3

Updated by Valentin Deniaud 14 days ago

Valentin Deniaud a écrit :

Aïe, gros fail de git-redmine (ma faute), j'ai envoyé dans master au lieu de soumettre le patch :'(

Pas de miracle, c'était rouge, j'ai revert, nouveau patch incoming.

#6

Updated by Paul Marillonnet 14 days ago

(Et en relisant la description du ticket j’ai l’impression que Fred ne voulait pas dire pour autant de retirer les jobs horaires, non ?)

#7

Updated by Frédéric Péters 14 days ago

(Et en relisant la description du ticket j’ai l’impression que Fred ne voulait pas dire pour autant de retirer les jobs horaires, non ?)

Les mises à jour restent effectuées dans le job daily; elles étaient juste présentes dans le hourly pour gagner du temps au début, ce qui n'est plus nécessaire avec job.

#8

Updated by Paul Marillonnet 13 days ago

  • Status changed from Solution proposée to Solution validée

Ok.

#9

Updated by Valentin Deniaud 7 days ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 82b9459cbf5bb8aaec6922ae2639fe197a60284a
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Wed Jan 13 10:41:16 2021 +0100

    base_adresse: use job to fetch data at create and edit (#50087)
#10

Updated by Frédéric Péters 7 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF