Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Affichage SMS "undefined" #17

Closed
ObiwanKennedyHep opened this issue Jan 2, 2017 · 5 comments
Closed

Affichage SMS "undefined" #17

ObiwanKennedyHep opened this issue Jan 2, 2017 · 5 comments

Comments

@ObiwanKennedyHep
Copy link

ObiwanKennedyHep commented Jan 2, 2017

Bonjour,

Quand je reçois le texto sur mon mobile, je n'ai pas l'image et pas d'habillage du texte. En gros j'ai le titre de l'annonce puis systématiquement "undefined" puis le prix (sans texte devant indiquant que c'est le prix; puis l'URL).
Ce serait pas mal d'habiller un peu plus le SMS, d'y ajouter l'image et de supprimer ce "undefined".

Autre remarque : Lors de l'envoi d'un SMS, la notification en bas à droite indique 'Mail envoyé à xxxx" mais ne précise pas si un texto est également envoyé.

Merci encore pour ton super travail !

Je précise que tous mes tests ont été réalisés avec la dernière version 5.2.4.

@maximelebreton
Copy link
Owner

Bonjour @ObiwanKennedyHep, le problème de undefined doit être résolu en 5.2.5.

Pour l'habillage du SMS, pourrais-je vous demander des impressions d'écran, car c'est une fonctionnalité expérimentale que j'ai commencé à développer à l'aveugle (n'ayant pas de compte free), donc je n'ai pas vraiment idée de ce qui s'affiche sur votre téléphone.

Merci !

@ObiwanKennedyHep
Copy link
Author

Voici un exemple !
(Le "hello World" c'est moi; pas le script).

Si besoin, je peux faire des tests avec vous si vous voulez.

img_4078

@maximelebreton
Copy link
Owner

Aaaah, merci !
Pouvez vous passer en 5.2.5 et rééssayer ? (le pb undefined devrait être résolu, et il devrait y'avoir un signe € pour le prix)

Qu'entendez vous par habillage de texte ? Car vu qu'il s'agit d'un SMS, les seules possibilités sont le texte brut.
D'ailleurs, vu qu'il s'agit d'un SMS, l'ajout d'image me semble impossible, sauf si l'API de free mobile autorise l'envoi de MMS, je vais me renseigner.

@ObiwanKennedyHep
Copy link
Author

ObiwanKennedyHep commented Jan 2, 2017

Oui en 5.2.5 le problème est résolu !
Ce que j'aurai imaginé est quelque chose comme ça :

Titre : %Titre%
Lieu : %location%
Prix : %price%
URL : %url%

avec donc un texte fixe qui annonce la variable et si possible un retour chariot.

Idéalement, la date et heure de parution de l'annonce serait top !
On peut également imaginer un "header" au début du 1er texto uniquement type :
===== Alerte LBC - %nom_alerte% =====

Comme tu le verras sur l'impression écran, il y a un bug avec l'encodage des caractères (apostrophe et &).

img_4079

URL recherche : https://www.leboncoin.fr/annonces/offres/ile_de_france/?th=1&q=deshumidificateur&it=1&location=Paris%2075012&parrot=0

@maximelebreton
Copy link
Owner

maximelebreton commented Jan 5, 2017

@ObiwanKennedyHep après vérification l'affichage d'une image n'est pas possible pour le moment car il s'agit de SMS, et non de MMS, et concernant les autres problèmes ou suggestions j'ai ouvert des issues spécifiques :
#19
#20

je ferme car l'issue principale est résolue.

maximelebreton added a commit that referenced this issue Feb 27, 2017
- fix Affichage SMS "undefined" #17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants