Project

General

Profile

Bug #13889

commande sync_external_feeds manquante

Added by Frédéric Péters about 8 years ago. Updated almost 8 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
Target version:
-
Start date:
07 November 2016
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:

Files

Associated revisions

Revision 1829923d (diff)
Added by Serghei Mihai about 8 years ago

add command for external rss feed synchronization (#13889)

History

#2

Updated by Serghei Mihai about 8 years ago

#3

Updated by Frédéric Péters about 8 years ago

timezone et CommandError importés pour rien.

Pas sûr qu'on ait envie de planter sur le premier fil indisponible.

#4

Updated by Serghei Mihai about 8 years ago

Ok, je passe toute exception levée.
Des cas comme flux RSS indisponibles doivent être gérés dans le save.

#5

Updated by Frédéric Péters about 8 years ago

Au moins "logguer" l'erreur.

#7

Updated by Frédéric Péters about 8 years ago

Je connais mal cette affaire mais a priori, le logger, je ne le chargerais pas à l'import du module, sur l'idée que le résultat peut peut-être être différent entre tenants.

#8

Updated by Serghei Mihai about 8 years ago

A priori c'est executé dans le contexte de chaque tenant, mais tu as raison, mieux vaut charger le logguer dans la méthode.

#9

Updated by Frédéric Péters about 8 years ago

Sans le continue inutile, ack.

#10

Updated by Serghei Mihai about 8 years ago

  • Status changed from En cours to Résolu (à déployer)

continue viré.

commit 1829923d105dcbfa14de5a4af46685bbc4690a27
Author: Serghei Mihai <smihai@entrouvert.com>
Date:   Mon Nov 14 16:27:58 2016 +0100

    add command for external rss feed synchronization (#13889)
#11

Updated by Serghei Mihai almost 8 years ago

  • Status changed from Résolu (à déployer) to Fermé

Also available in: Atom PDF