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.

Sign up for GitHub

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

Jump to bottom

Pas de parser YAML dans publicodes #602

Merged
merged 18 commits into from
Aug 23, 2022
Merged

Pas de parser YAML dans publicodes #602

merged 18 commits into from
Aug 23, 2022

Conversation

laem
Copy link
Contributor

@laem laem commented Aug 17, 2022 •
edited
Loading

PR miroir côté modèle : incubateur-ademe/nosgestesclimat#1451
démo https://publicodes-no-yaml--nosgestesclimat.netlify.app/?PR=1451

  • Les personas ne seront plus chargés que depuis la branche master. À corriger par la suite.
  • Avant de merger, enlever le bloc de code de EngineWrapper
  • fix Séparation des règles et du reste ! #504
  • MAJ publicodes <=46 : des problèmes
  • notamment quand on set un persona, peut-être du au changement de forme de la situation ? À vérifier, ça pourrait foirer toutes les situations
  • dépend de la mise en prod de No yaml dependency 2 publicodes/publicodes#254
  • faire un dernier check de l'appli
  • notamment la /documentation, avant de merger
  • inscrire dans la release draft
    Mission accomplie :)

image

laem added 11 commits August 17, 2022 16:20
@laem
TEMP on teste la nouvelle version de publicodes, sans le YAML, en local
f5adfb0
@laem
Chargement des personas via un JSON autonome des règles
d4d0c3c 
C'était sale, lourd et ça faisait planter publicodes v46
@laem
TEMP remove personas from rules
516624a
@laem
MAJ publicodes private rules v46
c182560
@laem
Déplacement du mécanisme de fetch des règles
a1b1c00
@laem
Chargement des personas via fichier en local ou via API
86aa315
@laem
Séparation des règles et des guides mode groupe
6dfd607
@laem
Séparation des règles et des règles plus
a814d4c
@laem
Encapsulation d'une nouvelle erreur publicodes intéressante
7f7b28f 
Je garde ces erreurs dans les peronas car elles sont représentatives de
ce qui arrivera à nos utilisateurs qui reviendront avec une ancienne
situation
@laem
Utilisation d'une version test de publicodes 47
f436635 
Je n'ai pas trouvé mieux que de la publier sur une URL NPM inconnue
@laem
NODE_ENV ne semble pas lu par webpack dans le hook
880b304
@laem laem mentioned this pull request Aug 22, 2022
Closed
5 tasks
@laem
Copy link
Contributor Author

laem commented Aug 22, 2022

  • il me semble y avoir un problème au premier chargement d'une page, qui se résoud au F5. Probablement que la bonne branche des données n'est pas téléchargée

@laem
Copy link
Contributor Author

laem commented Aug 22, 2022

J'ai tenté un fix, ça me semble marcher, je teste demain la tête fraiche

@Clemog
Copy link
Contributor

Clemog commented Aug 23, 2022

Pour le comportement de la fonctionnalité PR dans le menu l'idée est :

  • En mode dev : ne rien changer ?

  • Sur une branche du site en ligne : avoir l'indication de la branche du modèle sur laquelle on travaille / la faire disparaitre au clic sur la croix ?

@laem laem added this to En cours in Kanban Site Aug 23, 2022
@laem
Copy link
Contributor Author

laem commented Aug 23, 2022

Non, l'idée est qu'en mode dev on puisse aussi charger les règles d'une branche si PR=xx il y a.
Sinon, c'est les règles yaml en local.

@Clemog
Copy link
Contributor

Clemog commented Aug 23, 2022

Pourquoi est-ce important pour le mode dev ?
Changer de branche côté modèle est suffisant non ?

@laem
Copy link
Contributor Author

laem commented Aug 23, 2022

Ne serait-ce que pour tester la fonctionnalité elle-même :)

@laem laem merged commit 1648abc into master Aug 23, 2022
@laem laem deleted the publicodes-no-yaml branch August 23, 2022 14:50
@laem laem moved this from En cours to Fermé in Kanban Site Aug 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Clemog Clemog Clemog left review comments

Assignees
No one assigned
Labels
None yet
Projects
Kanban Site
Fermé
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Séparation des règles et du reste !
2 participants
@laem @Clemog