Projet

Général

Profil

Development #44833

Ajouter à l'import/export de site l'icône de l’application pwa

Ajouté par Mikaël Ates (de retour le 29 avril) il y a presque 4 ans. Mis à jour il y a plus de 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
06 juillet 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Lorsque l'on importe un site, les paramètres PWA sont importés dont la définition de l'icône PWA, mais le fichier de l'image n'est pas importé et il est facile d'oublier de le faire.


Fichiers

Révisions associées

Révision 95cb8060 (diff)
Ajouté par Nicolas Roche il y a plus de 3 ans

pwa: import/export pwa application-icon (#44833)

Historique

#1

Mis à jour par Frédéric Péters il y a presque 4 ans

Oui, pour les icônes de la navigation on a déjà du code dédié,

        if self.icon:
            serialized_entry['icon:base64'] = force_text(base64.encodebytes(self.icon.read()))
#2

Mis à jour par Nicolas Roche il y a plus de 3 ans

Je me demande si ce ne sera pas redondant avec https://dev.entrouvert.org/issues/45128 (qui copie tous ce qu'il y a dans default_storage).

Surtout le chargement du fichier d'import est trés long (22 secondes) lorsqu'il contient une icône de 12K.

#3

Mis à jour par Frédéric Péters il y a plus de 3 ans

Surtout le chargement du fichier d'import est trés long (22 secondes) lorsqu'il contient une icône de 12K.

Par rapport à combien de temps sans le patch ?

#4

Mis à jour par Nicolas Roche il y a plus de 3 ans

Sans le patch, l'import du fichier joint (export fait sans le patch) prend tout autant de temps (22 secondes),
en fait c'est n'est absolument pas lié au fait que l’icône soit ajoutée dans le json.
(désolé pour la fausse info).

#5

Mis à jour par Frédéric Péters il y a plus de 3 ans

  • Statut changé de Information nécessaire à Solution proposée
#6

Mis à jour par Frédéric Péters il y a plus de 3 ans

  • Statut changé de Solution proposée à En cours
  • Patch proposed changé de Oui à Non

Ça va planter quand il n'y a pas d'icône, façon

ValueError: The 'application_icon' attribute has no file associated with it.

#7

Mis à jour par Nicolas Roche il y a plus de 3 ans

Oui (du coup j'ai ajouté ce test).

#8

Mis à jour par Frédéric Péters il y a plus de 3 ans

  • Statut changé de Solution proposée à Solution validée
#9

Mis à jour par Nicolas Roche il y a plus de 3 ans

  • Statut changé de Solution validée à Résolu (à déployer)
  • Assigné à mis à Nicolas Roche
commit 95cb806044c244bd494e0a123e00a41ccd214b5c (HEAD -> master)
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Thu Jul 30 19:06:58 2020 +0200

    pwa: import/export pwa application-icon (#44833)
#10

Mis à jour par Frédéric Péters il y a plus de 3 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF