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

[v14] Autocompletion des membres dans les choux #3192

Closed
Eskimon opened this issue Nov 13, 2015 · 18 comments
Closed

[v14] Autocompletion des membres dans les choux #3192

Eskimon opened this issue Nov 13, 2015 · 18 comments
Assignees
Labels
C-Front Concerne l'interface du site S-Régression Corrige un problème sur un composant qui fonctionnait auparavant
Milestone

Comments

@Eskimon
Copy link
Contributor

Eskimon commented Nov 13, 2015

Sur la v14 l'autocompletion des membres est complètement dans les choux, il n'est plus possible de sélectionner un pseudo et c'est très gênant. J'ai testé ca sur la création de MP ou l'ajout d'un auteur à un tuto et on a les mêmes symptômes. Ce problème n'existe pas sur la prod'.

@Eskimon Eskimon added S-Régression Corrige un problème sur un composant qui fonctionnait auparavant Bloquant Ticket qui doit être traité avant la prochaine mise à jour labels Nov 13, 2015
@pierre-24
Copy link
Member

@GerardPaligot : l'API des membres ?

@artragis
Copy link
Member

Info technique : la requête ajax se prend une 403.

@Eskimon
Copy link
Contributor Author

Eskimon commented Nov 13, 2015

Info technique : la requête ajax se prend une 403.

Probablement chez toi car l'API est bloqué de ton côté. Par contre chez moi ca marche pas quand même.

@Eskimon
Copy link
Contributor Author

Eskimon commented Nov 13, 2015

Normalement l'API des membres est accessible par tous tout le temps sur la beta (et l'API des MP soumise à filtre IP, et le reste du site soumis au htpass)

@Eskimon
Copy link
Contributor Author

Eskimon commented Nov 13, 2015

Bon apparemment c'est possible que ce soit le front qui soit dans les choux vu qu'on a plus d'icones pour l'éditeur de texte... je vais refaire la release en logguant bien tout je pense...

@Eskimon
Copy link
Contributor Author

Eskimon commented Nov 13, 2015

RAS sur le déploiement, je vois pas d'où vient le problème...

@GerardPaligot
Copy link
Member

On sait me rediriger vers la requête front qui est faite sur l'API ?

@Situphen
Copy link
Member

Je viens de tester sur la béta et en effet l’auto-complétion est morte, mais ça a l'air d'être l'affichage et non pas les requêtes (qui ont un code 200). Je ne sais pas pourquoi car normalement ni le code JS ni les vues ont été changées. Par ailleurs, les smileys aussi ne s'affichent plus alors qu'ils sont encodés en base64 dans le code (S)CSS. Bizarre bizarre cette histoire !

Pour information, le fichier JS gérant l'auto-complétion est ici.

@GerardPaligot
Copy link
Member

Bon, si le code retourné est un code 200. Inutile pour moi de regarder plus loin. :)

@GerardPaligot
Copy link
Member

J'ai testé l'autocompletion et on peut clairement se dire que ce n'est pas un problème d'API. La recherche est faite et renvoyée au front mais elle n'est pas bien gérée. On a un gros bug d'affichage. J'ajoute le tag front.

@GerardPaligot GerardPaligot added the C-Front Concerne l'interface du site label Nov 14, 2015
@artragis
Copy link
Member

Je pense que c'est la génération du front qui a foiré.

@pierre-24
Copy link
Member

Qu'est ce qui te fait dire ça ?

@artragis
Copy link
Member

l'éditeur de texte sans image. et deux trois trucs qui ne fonctionnent
plus dans le js alors que personne n'y a touché.

Le 21/11/2015 10:01, Pierre Beaujean a écrit :

Qu'est ce qui te fait dire ça ?


Reply to this email directly or view it on GitHub
#3192 (comment).

@Eskimon
Copy link
Contributor Author

Eskimon commented Nov 21, 2015

Je déploie une RC3, on verra bien si ca corrige le souci

@Eskimon
Copy link
Contributor Author

Eskimon commented Nov 21, 2015

Bon le front semble un peu mieux marcher, mais l'auto-complétion n'est toujours pas revenu comme elle le devrait.

@GerardPaligot
Copy link
Member

0 intervention de développeurs fronts. Pour avancer, ça me semble nécessaire.

ping @sandhose et @Situphen (@Alex-D :-°)

@Fulbert
Copy link
Contributor

Fulbert commented Nov 24, 2015

Chez moi les noms s'affichent mais ils ne sont pas cliquable. Les requêtes fonctionnent bien sur des URL vers https://beta.zestedesavoir.com/api/membres/?search=eski (200).

@sandhose
Copy link
Contributor

note: si on me mentionne pas, je risque de pas voir ce genre de tickets

Donc, le problème vient du faire que la réponse côté serveur d'un coup s'est transformée de pk en id

Forcément, comme c'est ce champ qui était utilisé pour la sélection, ça marche moins bien .-.

PR ouverte: #3210

@gustavi gustavi removed the Bloquant Ticket qui doit être traité avant la prochaine mise à jour label Nov 25, 2015
@gustavi gustavi closed this as completed Nov 25, 2015
@gustavi gustavi added this to the Version de développement milestone Nov 25, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Front Concerne l'interface du site S-Régression Corrige un problème sur un composant qui fonctionnait auparavant
Projects
None yet
Development

No branches or pull requests

8 participants