• upload des images resizées sur le s3
• corriger les routes des images dans le HTML

exemple d'implémentation en PHP

https://badsender-test.herokuapp.com/editor/579eff96f823b40300fc4b5f

TypeError: Cannot read property 'mosaico' of null
    at /app/server/creations.js:44:61
    at process._tickDomainCallback (internal/process/next_tick.js:129:7)

• reproduce
• patch
• deploy dev
• test badsender
• deploy prod
• test badsender prod

Actuellement si on veut vérifier le mailing il n'y que la possibilité de se l'envoyer par mail, ou de télécharger le fichier html.

Si on veut pouvoir sauvegarder toutes les images associées à une création on est obligé de :

• télécharger le fichier HTML
• l'ouvrir dans un navigateur
• faire fichier -> enregistrer sous
• et ainsi accéder à la liste complète des images

Il pourrait être utile de :

• faire une pré-visualisation dans le navigateur
• faire fichier -> enregistrer sous
• et ainsi accéder à la liste complète des images

En vue de maîtrisé les coûts, il serait intéressant de pouvoir surveiller l'application afin de pouvoir optimiser les besoins de ressources de cette dernière

• forcer le https si option serveur activée
• corriger les urls d'images de mosaico en edres
• test dev
• test stage

pour tester

Toute url en http devrait être redirigée en https

par exemple :

• http://badsender-test.herokuapp.com/?page=1 devrait renvoyer vers https://badsender-test.herokuapp.com/?page=1
• http://badsender-test.herokuapp.com/cover/258x100/57c28bf85eeda60a009af548-llama_2.jpg devrait renvoyer vers https://badsender-test.herokuapp.com/cover/258x100/57c28bf85eeda60a009af548-llama_2.jpg

description du problème

Quand le site est en https, l'éditeur ne fonctionne plus et/ou mal

Failed to load resource: net::ERR_INSECURE_RESPONSE

In the generated HTML, special characters are encoded, it is a problem for some of them when we need to use ESP tags.

By example "Hello <% FIRSTNAME %>" is transformed into "Hello < FIRSTNAME >" when exported.

• Create a simple test case
• Make an automated test
• Deploy test server
• Test Badsender
• Deploy prod
• Test Badsender prod

Functional tests should be provided to avoid any regression as the application grow.

• https://github.com/theintern/intern
• https://www.npmjs.com/package/nightmare

Hello,

Another problem linked to the ESP issue is the way the images are called. They are not directly called in the src code, instead they are called via parameters, and Adobe Neolane can't download the images in its system.

A workaround would be to be able to download a zip with the HTML and all the images and with the path to images in a relative path (like images/my_image.jpg)

Sera utilisé pour la mise en productions des comptes companies

add user ip address

https://lostechies.com/derickbailey/2013/12/04/getting-the-real-client-ip-address-on-a-heroku-hosted-nodejs-app/

Idem à #22 mais pour la partie admin :

• utilisateurs pas possible de supprimer un utilisateur
• wireframes
• sociétés pas possible de supprimer une société

Hello,

We would like to be able to edit the email name not only in the home page but also on the email creation page

• Show actual creation name in the editor
• On double click, convert to input
• On ok (button)
  • save to DB
  • revert to plain text
• On cancel (button)
  • revert to previous name
  • revert to plain text
• deploy on dev
• badsender validation
• deploy on prod
• test prod badsender

Est-il possible de pouvoir choisir en entrant manuellement la taille de la typo?

https://devcenter.heroku.com/articles/github-integration-review-apps

Pour résoudre les tickets #18 et #20

• Étapes de dev :
  • Interface de création des sociétés
  • Interface de migration d'un utilisateur
  • Modification home utilisateur : un utilisateur voit tout les templates et les créations de sa société (lié à #19)
  • Script de copie DB prod -> DB dev
  • Script de copies DB stage -> DB prod (#44)
  • Script de backup de DB prod
  • Création des pages de maintenance/erreur (#26)
  • Corrections retours Grégory
• déploiement dev :
  • Test avec Badsender d'une migration à partir de la DB de dev
• déploiement stage :
  • Test avec Badsender d'une migration à partir d'un copie DB de la prod
  • Mode maintenance de l'application prod (lié à #26)
  • Modification depuis le stage d'une copie de la DB prod
  • Déploiement de la DB modifiée sur la prod
  • Déploiement code en production
  • Enlever le mode maintenance

Ajouter l'auteur d'une création dans le tableau listing

• corriger #33
• ajout de la fonctionnalité sur mosaico
• test dev
• test stage
• test prod

Un utilisateur ne doit plus être suprimable, mais désactivable.

En effet, s'il est supprimé les infos de compte seront perdu et les templates qu'il aura crée n'auront plus accès à son nom

• comportement
  • user désactivable depuis l'admin
  • créations préservées
  • impossible de se connecter sur un user désactivé
• test dev
• test prod

On devrait reverser à Mosaico ce commit là

#7

• des images ayant le même nom mais pas redimensionné à l'identique (confirmé : #61)
• des images en background-img: url(...) dans la balise ou l'attribut style

Adobe campaign should be able to download all the assets from a generated mailing

• test sushishop
• correction de l'issue #32
• test Sushishop depuis le dev
• test avec le client d'un zip avec les caractères spéciaux encodés
• réponse consultant logs d'erreurs
• test shushishop depuis la prod suite au déploiement de #32 (meilleur temps de réponse ?)

Les pages devraient

• être aux couleurs de badsender
• pouvoir fournir des informations utiles aux utilisateurs.
  Par exemple, un email ou téléphone de contact

• Création de la page d'erreur
• Création de la page de maintenance
• Validation des templates par BS
  • erreur
  • maintenance
• Déploiement sur un S3
• Configuration des instances :
  • dev
  • stage
  • production

Cela permettra de tirer tout les avantages du scale d'heroku par le futur

https://devcenter.heroku.com/articles/node-concurrency

• deploy dev
• test badsender
• deploy stage
• test badsender

documentation :

• heroku free ssl
• heroku ssl beta
• heroku old ssl endpoint
• gandi ssl

À partir du template “tutorial badsender”

En console.warn

app.js:101 Missing translation string for Fake image editor : using default string
app.js:101 Missing translation string for <p>Fake image editor</p> : using default string
app.js:101 Missing translation string for Creation has been saved : using default string

En console.log

editor.js:181 TODO WARN Missing label for object  template
editor.js:181 TODO WARN Missing label for object  HeaderAndText
editor.js:181 TODO WARN Missing label for object  HeaderAndText
editor.js:204 TODO missing label for theme section  undefined HeaderAndText
editor.js:181 TODO WARN Missing label for object  buttonBlock
editor.js:181 TODO WARN Missing label for object  buttonBlock
editor.js:204 TODO missing label for theme section  undefined buttonBlock
editor.js:178 TODO WARN Missing label for property  backgroundColor
editor.js:181 TODO WARN Missing label for object  fixedlist
editor.js:178 TODO WARN Missing label for property  listsize
editor.js:181 TODO WARN Missing label for object  fixedlist
editor.js:204 TODO missing label for theme section  undefined fixedlist
editor.js:178 TODO WARN Missing label for property  color
editor.js:181 TODO WARN Missing label for object  theme
editor.js:178 TODO WARN Missing label for property  bodyTheme
editor.js:178 TODO WARN Missing label for property  backgroundColor
editor.js:178 TODO WARN Missing label for property  color

http://docs.mlab.com/backups/

• nom email
• ajout de la date

Les email téléchargés devraient avoir pour nom le titre de la création + la date génération

Currently, a set of Master Templates and created email is linked to a single user. It is not possible to share Master template or created emails between users.

We would like to be able to create User groups (who are going to be companies). Inside these user groups, the users need to be able to share the master templates and see/edit/duplicate the created emails of each others.

Thanks

• merge mosaico
• création d'un template de test avec les commentaires conditionnels
• correction du code de mosaico

microsoft documentation

La fonctionnalité attendue est la suivante:
1 compte société au sein duquel existent des comptes utilisateurs.

• Les templates de la société sont commun à tous les utilisateurs de la société
• Le super-admin peut créer des utilisateurs pour chaque société
• Pas de password-less connection
• Les créations sont communes à tous les utilisateurs de la société

Remarques de @Hiswe

• enlever la possibilité pour une société de se connecter
• Un compte société est directement activé
• Quid de création existante ?

--> À voir au cas par cas. Il faudra probablement fusionner certains comptes.

Suite au #45, il faudra procéder à la suppression du code de migration, ainsi que du code utilisé uniquement dans l'ancien type d'organisation

• nettoyage du code
• test dev
• test stage

Ajouter un message demandant confirmation lors de la suppression d'une création

• deploy dev server
• test badsender
• deploy stage server
• test badsender

La home page d'un compte société ne peut être accessible que par un super admin.

Sur cette page:

• Voir la liste des membres de la société
• Pouvoir en ajouter/éditer un utilisateur laisser langue par utilisateur
• Voir la liste des création de la société et possibilité de les éditer ou d'en créer

Ajout d'un environnement de pre-prod afin de pouvoir tester les nouvelles fonctionnalités dans un environnement qui utilise déjà les tables de production.

• Création du stage
• Création du script de déploiement du stage
• Test badsender (vérification que le stage est identique à la prod)

• fix problem
• deploy test
• test badsender

badsender-lib-core.js:10424 Uncaught ReferenceError: Unable to process binding "block: function (){return $data }"
Message: Unable to process binding "template: function (){return (typeof templateMode !='undefined'&& templateMode =='wysiwyg') || image().url() ?'anonymous-89019':'anonymous-14926' }"
Message: Unable to process binding "template: function (){return 'anonymous-14926' }"
Message: Unable to process binding "wysiwygImg: function (){return { _data:$data,_item:image,_template:'anonymous-73956',_editTemplate:'img-wysiw…rc,_width:imageWidth(),_height:null,_align:undefined,_size:'w'+imageWidth() +'',_method:undefined,_placeholdersrc:{ width:imageWidth(),height:150,text:'w'+imageWidth() +''},_stylebind:{ width:imageWidth()}} }"
Message: Unable to process binding "template: function (){return _template }"
Message: Unable to process binding "wysiwygSrc: function (){return { width:imageWidth(),height:null,src:image().src,placeholder:{ width:imageWidth(),height:150,text:'w'+imageWidth() +''}} }"
Message: options is not defined

• Modifications à effectuer
  • modification du routing des images serveur & mosaico
  • cohabitation avec l'ancien code (1 mois)
  • renommage des images
• test Badsender Dev
• test Badsender Stage
• test Badsender Prod
• déploiement en production coordonné avec Badsender (réupload des images de preview edres)

Ce qui est attendu

http://builder.badsender.com/img/?method=resize&params=440%2Cnull&src=%2Fimg%2F57767da32f037e03000a7bd7-video.jpg

devrait être :

http://builder.badsender.com/img/resize/440xnull/2F57767da32f037e03000a7bd7-video.jpg

ainsi que renommer (url friendly) les images uploadées

Bien faire attention au path des images une fois le mailing téléchargé

Problème actuel relevé

Exemple d'url qui ne fonctionne pas (voir #29 (comment)) :

http://builder.badsender.com/img/?method=resize&params=440%252Cnull&src=%252Fimg%252F57767da32f037e03000a7bd7-tyrsa.jpg

Il semble que l'url est encodée 2 fois :

var url = 'http://builder.badsender.com/img/?method=resize&params=440%252Cnull&src=%252Fimg%252F57767da32f037e03000a7bd7-tyrsa.jpg';
var decodedUrl = decodeURIComponent(url);

http://builder.badsender.com/img/?method=resize&params=440%2Cnull&src=%2Fimg%2F57767da32f037e03000a7bd7-tyrsa.jpg

var redecodedUrl = decodeURIComponent(decodedUrl);

http://builder.badsender.com/img/?method=resize&params=440,null&src=/img/57767da32f037e03000a7bd7-tyrsa.jpg

voir #2 (comment)

• test avec sendgrid en local
  • envoie sur mail app (MacOs)
  • envoie sur google mail (client web)
• test avec sendgrid en dev
  • envoie sur mail app (MacOs)
  • envoie sur google mail (client web)
  • envoie sur https://www.emailonacid.com (badsender)
• configuration sendgrid en prod
• test badsender

Sur la page de listing des emails:

• pagination
• tri
• Sortir le listing des créations dans une page propre (home)
• Créer une page de nouvelle création à partir d'un wireframe
• filtres (nom, date de création rechercher)

• maquettes
• suppression du bloc “en savoir plus”
• enlever les liens de filtre sur le tableau
• ajouter une croix de suppression des filtres quand filtre actif et fermé
• pagination en 10, 25 et 50
• ajout des fonctionnalités :
  • tri (nom, utilisateur, wireframe, dates)
  • filtres (nom contient, dates de création/update, utilisateurs, wireframes)
• déplacement du logo badsender (en vecto dans le header)
• test dev
• test stage

Lors du choix de la taille de typo, l'unité doit être en px et non en pt

• change to 8px 10px 12px 14px 18px 24px 36px
• deploy dev
• deploy prod
• test prod badsender

• add he on backend
• don't encode tabs
• test dev

https://github.com/mathiasbynens/he

Emails should be send when app is down.

https://blog.heroku.com/threshold-alerting-generally-available

Problème technique dont la résolution nous éloigne de la base de code de Mosaico

Il faudrait pouvoir permettre à Mosaico de nous laisser la possibilité de remplacer la fonction _templateUrlConverter de manière simple

voidlabs#181

Hello,

We would like to update the toolbar that appear when we edit a text:

1. We would like to remove the "Headings" and "Blocks" options in the format option:
   

2. We would like to be able to choose more color from the color picker (either on the text color or background text color). Ideally, with to possibilities for the user to either use a more evovlved color picker and a possibility to enter the hexadecimal value for a color.
   So instead of this,
   
   Having something like this:
   

3. We would like to have the possibility to change the font size in the toolbar.

• remove the "Headings" and "Blocks" options in the format option
• choose more color from the color picker (either on the text color or background text color).
• change the font size
• Mettre sur le dev
• Test Badsender
• Deploy prod || voir aussi #15 (changement taille pt en px)
• test prod badsender

Hello,

Lors de l'utilisation d'une image qui est un GIF animé, le builder semble complètement perdre les pédales, ce qui rend l'import de GIF animé impossible.

Le test peut-être fait avec le GIF animé ci-joint:

Le problème se pose dans tous les templates, même ceux de Mosaico.

• test 2 gifs
• correction du problème
• deploy test
• test badsender
• deploy prod
• test prod badsender

Sometimes the text bar stays on the screen even when we click somewhere else and scroll elsewhere.

Issue can sometimes be reproduced by

1. click in an editable text
2. click on another block (but not an element in a block)

This issue can be marked as low prio

• Find a way to always reproduce
• patch
• deploy dev
• test badsender dev
• deploy prod
• test badsender prod

• test dev
• test stage

Hello,

Il semble y avoir un problème d'Overlay dans le picker de couleur du menu de droite sur le builder:

Il semblerait qu'ajouter un attribut z-index:1000 à la classe CSS .mo .evo-pop résolve le problème.

Logs sans et avec resize :

• connect=0ms service=347ms
• connect=0ms service=2219ms

2016-08-30T10:02:01.321301+00:00 heroku[router]: at=info method=GET path="/img/57c5509994f5380a00b3c439-gif_anime_lunchbox_juillet_espagne.gif" host=badsender-stage.herokuapp.com request_id=1ad69608-12fc-4dc4-83cf-a9d6cd28c935 fwd="180.183.159.173" dyno=web.1 connect=0ms service=347ms status=200 bytes=430117
2016-08-30T10:02:21.437061+00:00 heroku[router]: at=info method=GET path="/resize/600xnull/57c5509994f5380a00b3c439-gif_anime_lunchbox_juillet_espagne.gif" host=badsender-stage.herokuapp.com request_id=b62dd7e3-da98-43d0-8c86-163cdfe249f8 fwd="180.183.159.173" dyno=web.1 connect=0ms service=2219ms status=200 bytes=994257

–––

edit 2 septembre

Message d'alerte envoyé par Heroku suite au redimensionnement d'un GIF. 11 secondes…

2016-09-01 10:16:51.245
370 <158>1 2016-09-01T10:16:50.807493+00:00 heroku router - - at=info method=GET path="/resize/600xnull/57c7ffd91c4b3e0a005b0784-gif_anime_lunchbox_juillet_espagne.gif" host=badsender-stage.herokuapp.com request_id=230dfc04-d679-43fa-b360-b3ad70d600a8 fwd="109.128.77.12" dyno=web.1 connect=1ms service=11740ms status=200 bytes=994800

edit 23 septembre

• ne pas resizer un gif s'il est déjà à la bonne taille #76
• test dev
• test stage
• controle :
  • cache côté client