Skip to content

Météo : signaler que Météo Concept couvre la France seulement - #50

Open
LucQuebec wants to merge 1 commit into
lfpoulain:betafrom
LucQuebec:proposition/meteo-hors-france
Open

LucQuebec wants to merge 1 commit into
lfpoulain:betafrom
LucQuebec:proposition/meteo-hors-france

Conversation

@LucQuebec

Copy link
Copy Markdown

Ferme #47, sous la forme retenue dans le ticket.

Ce qui change

  • Administration, carte « Météo » : sous la description, « Météo Concept couvre la France seulement. Ailleurs, installez Météo libre (Open-Meteo, sans jeton) depuis Installer des modules. », avec un lien vers la rubrique #catalogue.
  • Documentation : la fiche du module Météo (introduction et « Limites »), le tableau des modules de docs/modules/README.md et la page de l’administration (docs/utilisation/administration.md) donnent la même précision.
  • CHANGELOG.md : une entrée sous « Non publié ».

Vérifié

  • npm test : 804 réussis sur 804, sur beta.
  • Tests du navigateur : admin-options.spec.ts vérifie la phrase et le lien #catalogue dans la carte Météo ; addons.spec.ts cherche désormais « Météo Concept » en texte exact (la nouvelle phrase contient aussi ces mots). Les deux fichiers : 4 tests réussis, après npm run build.

Luc, Jokobee™

La carte « Météo » de l'administration précise que Météo Concept couvre la
France seulement et renvoie, ailleurs, vers Météo libre (Open-Meteo, sans
jeton), avec un lien vers « Installer des modules ». Même précision dans la
fiche du module et la documentation des modules. Test du navigateur : la
phrase et le lien ; un test existant cherche désormais « Météo Concept » en
texte exact.

Refs lfpoulain#47
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

Successfully merging this pull request may close these issues.

1 participant