Options de mise à jour : lien vers changelog

[MrWaloo]

Le lien ne fonctionnait pas (cf sujet du community)
Avec cette PR, le lien ressemble à ce qui est fait dans la table des mises à jour ET dans la fenêtre "à propos".
En PHP, le thème n'est pas accessible comme ce serait le cas en JS, puisqu'il n'est pas possible de relire document.body.getAttribute('data-theme') comme en JS. J'ai donc préféré supprimer le thème du lien plutôt que d'écrire tout une usine à gaz.

$theme = 'light';
if (strpos(config::byKey('jeedom_theme_main'), 'Dark') !== false) {
	$theme = 'dark';
}

n'est pas suffisant pour détecter le thème alternatif dans Jeedom.

Description

Suggested changelog entry

Options de mise à jour : lien vers changelog

Related issues/external references

Fixes #

Types of changes

• Bug fix (non-breaking change which fixes)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
  • This change is only breaking for integrators, not for external standards or end-users.
• Documentation improvement

PR checklist

• I have checked there is no other PR open for the same change.
• I have read the [La ligne directrice pour contribuer à ce projet / Contribution guidelines for this project).
• I grant the project the right to include and distribute the code under the GNU.
• I have added tests to cover my changes.
• I have verified that the code complies with the projects coding standards.
• [Required for new sniffs] I have added MD documentation for the sniff.

[Mips2648]

Salut,
pourquoi ne pas le faire en JS comme c'est justement le cas dans la fenêtre about?

[MrWaloo]

C'est finalement ce que j'ai fait et j'ai même poussé jusqu'à afficher le changelog de la nouvelle version, même en cas de mise à jour majeure (4.5 -> 4.6 par exemple).

[MrWaloo]

Est-ce que j'ai quelque chose à faire pour l'étiquette changelog-fix ?
J'avais proposé un texte pour le changelog dans la description initiale. Il ne convient pas ?

[Mips2648]

Salut,
non rien à faire concernant le label changelog-fix ni pour l'entrée dans le changelog, c'est bien comme ça (ou on reverra ca plus tard si besoin).

Concernant la PR, elle est assignée à @Salvialf et c'est voulu, il voulait la revoir
Donc pas d'inquiétude, elle n'est pas oubliée et elle sera prise en compte

[MrWaloo]

Merci pour cette réponse très claire.