Toutes les notification renvoi sur la mauvaise url

[Eragonfr]

Le bouton "toutes les notifications renvois vers l'url https://zestedesavoir.com/forums/notifications/
alors qu'il devrai renvoyer vers https://zestedesavoir.com/notifications/
une url est mal écrite dans le code
Je suis avec le navigateur vivaldi et opera, je n'ai pas testé avec firefox

[AmarOk1412]

Euh, quelle version et quelle cocde d'extension ? Google Chrome et Firefox envoient vers la bonne url.

[viki53]

Je confirme que j'ai aussi ce bug sur la version Chrome (1.2.3, juin 2015).

Le store ne propose pas de version plus récente. Faudrait voir avec @Eskimon pour pousser la dernière version.

[AmarOk1412]

Le code de google chrome contient déjà la bonne url en plus. Weird :)

Sinon y a toujours cette PR #46

[viki53]

Oui mais il est pas mis en prod je pense.

Faudrait voir pour finaliser la PR, tant qu'à faire, toutes les versions seraient alignées…

[AmarOk1412]

Ouai faut que je finisse cette PR, mais faudrait que j'assigne quelqu'un pour test

[Eskimon]

Mon souci est que je n'utilise plus chrome, et du coup je ne test plus ce notificateur. Quelqu'un de confiance par là voudrais reprendre le bébé ?

[Eragonfr]

désolé de mon inactivité, j'ai la version 1.2.3 de l'extension et pour moi l'url est mauvaise
Vivaldi utilise les extensions de chrome et le moteur de rendu chromium
j'ai vérifié la disponibilité d'une nouvelle version et non l'url n'est pas bonne dans la version sur le chrome store
je vait vérifier le code qu j'ai mais c'est le même que sur le store normalement

[viki53]

@Eskimon Je peux si tu veux. Mais ça implique de changer le compte qui publie.

Je peux utiliser le compte de ZdS remarque, vu que je suis dans l'équipe comm'…

[Eskimon]

Ah bah carrément oui. Il y a un moyen de changer la paternité du truc dans les interfaces google ?

[viki53]

Aucune idée !

[Eragonfr]

//ligne "Afficher toute les notifications"
		if(notificator.getOptions("showAllNotifButton")) {
			var all_notifs_link = document.createElement("a");
			all_notifs_link.classList.add("element", "other", "allNotifs");
			all_notifs_link.href = urlZdS + "/forums/notifications";
			all_notifs_link.textContent = "Toutes les notifications";

			content.appendChild(all_notifs_link);
		}

voilà la partie du code à remettre en question chez-moi

[Eragonfr]

Je me suis permit de le modifier en

//ligne "Afficher toute les notifications"
		if(notificator.getOptions("showAllNotifButton")) {
			var all_notifs_link = document.createElement("a");
			all_notifs_link.classList.add("element", "other", "allNotifs");
			all_notifs_link.href = urlZdS + "/notifications"; \\ changé !!
			all_notifs_link.textContent = "Toutes les notifications";

			content.appendChild(all_notifs_link);
		}