Liens doc et changelog stable KO

Plugins Météo
1

Bonjour,

Tout est dans le titre.
Merci pour votre plugin.

PS : ce bouton aussi est KO :
image

2

Hello
Normal et Bison l’a bien annoncé ici et il y a d’ailleurs les Changelog

Soyez indulgent quand un plugin sort, et n’hésitez pas à faire des recherches avant

3

Salut,

Ah, mince, j’ai du me rater dans le nom dans la page puisque je l’ai bien faite en stable.
Je regarde ça en rentrant pour corriger, merci pour l’info.

@yoyouri, merci pour ton intervention mais le commentaire que j’avais fait était pour la version beta lors du lancement du plugin.
Actuellement tout devrait être OK (sauf le lien de la doc de la version bêta car il faut que je trouve comment identifier la version pour renvoyer vers le bon lien) donc il y a bien un truc qui ne va pas.

4

Je suis indulgent, ce n’était en aucun cas une critique mais simplement pour le signaler puisque le plugin est passé en stable.
C’est simplement que le lien pointe là :
https://github.com/BisonJeedom/documentations/blob/main/blitzortung/index_stable.md
Au lieu de là :

Merci.

5

dans le info.json

	"changelog": "https://...",
	"documentation": "https://.../",
	"changelog_beta": "https://m...elog",
	"documentation_beta": "https://mip.../",

dans le code, en php:

$plugin->getDocumentation()
2 « J'aime »
6

Je re-check le code en arrivant mais de tête c’est bien ce que j’ai fais

2 « J'aime »
7

Presque :slight_smile:

	"documentation": ".../blitzortung/index_stable.md",
	"documentation_beta": ".../blitzortung/index_beta.md",

Le fichier stable existe dans le suffixe _stable

8

Yes je me suis raté hier soir en publiant index.md et non index_stable.md comme je l’avais prévu. J4ai été trop vite.

Mais là je parlais du bouton documentation que j’avais rajouté dans le plugin pour un accès direct à la documentation et qui ne fonctionne pas dans la version beta (il ne se passe rien en cliquant dessus alors que ça devrait ouvrir index_beta.md).

Bon … en même temps si ça se trouve même en stable il ne fonctionne pas mais pour le moment je ne sais pas si ça vient du mauvais nommage ou du code :slight_smile:

9

Sur la page principale du plugin, ya des chances que tu n aies rien a faire si ce n est mettre les bonnes classes pr reutiliser le js du core !? (Et donc ne pas t inquieter du « beta » vs « stable »)

10

Je te confirme, hs aussi en stable :wink:

11

Ok merci … bon ben j’ai du me rater dans le code et j’ai cru que c’était du au fait qu’il récupérait systématiquement le nom de la version stable, qui n’existait pas encore.

Bon je regarde dès que je peux et je repasse ici avoir un peu d’aide si je ne trouve pas mais comme j’avais déjà fait ça sur un autre plugin ça m’étonne que je me sois raté mais à l’évidence c’est le cas.

12

En regardant le code, tu n’as pas d’ addEventListener dans ton js donc aucun évènement sur le bouton.

1 « J'aime »
13

OK merci :slight_smile:

14

Voilà donc j’ai corrigé le nom du fichier stable et ajouté le bout de js qui manquait effectivement pour que le bouton réponde au clic.

Il reste le problème que j’avais vu au départ (et ce pourquoi je disais qu’il ne récupérait pas la bonne URL quand j’étais qu’en bêta) :

			<div class="cursor pluginAction logoSecondary" data-action="openLocation" data-location="<?= $plugin->getDocumentation() ?>">
				<i class="fas fa-book-open"></i>
				<br>
				<span>{{Documentation}}</span>
			</div>

Je suis sur la branche beta :
image

Mais en passant la souris sur l’icône, on voit que la fonction $plugin->getDocumentation() récupère l’adresse de la doc stable et non bêta.

Du coup j’ai toujours pas compris, Mips tu es sur que c’est sensé marcher comme ça rien qu’avec cette fonction ?

image
image1198×578 34.9 KB

1 « J'aime »
15

En faite en beta, il faut utiliser getDocumentation_beta()

16

J’étais en train de regarder le code du core pour essayer de comprendre comment ça marche. Mais s’il faut utiliser une autre fonction alors ma question reste la même : « il faut que je trouve (retrouve) comment identifier la version pour renvoyer vers le bon lien », je vais regarder du coté de getSource()

17

Allez kdo…

public static function isBeta($text = false) {
        $plugin = plugin::byId('xxx');
        $update = $plugin->getUpdate();
        $isBeta = false;
        if (is_object($update)) {
            $version = $update->getConfiguration('version');
            $isBeta = ($version && $version != 'stable');
        }

        if ($text) {
            return $isBeta ? 'beta' : 'stable';
        }
        return $isBeta;
    }
1 « J'aime »
18

Ah, c’est pas getSource, ça donne « Market » :slight_smile:

Merci pour la fonction :grin:

19

Par contre… je ne vois pas le lien community…???
:open_mouth:

https://community.jeedom.com/t/creer-un-post-community-quasi-automatiquement/109880?u=tomitomas

1 « J'aime »
20

tu ne la pas encore PR dans la config du plugin :wink: