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.

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

Fix GitHub pages #4

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

totakoko
Copy link

@totakoko totakoko commented Sep 5, 2024

Hello !
Voici quelques corrections suite à l'utilisation côté FCU. Pas mal de trucs ne marchaient pas concernant les liens directs vers github pages.

Maintenant sur FCU, on peut faire des liens vers les règles directement. => Cad être sur la page d'une règle et recharger la page sans perdre la page.

C'est une PR qui comprend plusieurs corrections, tu pourras voir chaque commit séparément.

Pour la partie préfixe (model-template), qu'il faut remplacer, j'ai dû ajouter qqch dans vite.config.ts, c'est vraiment pas génial et ça serait mieux que le préfixe soit juste configuré au build via variable d'env par exemple, et donc bien spécifié en CI pour github pages. Là c'est le plus rapide qui a été fait mais le plus cohérent avec l'existant.

Dispo si t'as des questions !

@EmileRolley
Copy link
Collaborator

Hello @totakoko, merci pour la PR ! Je viens de la voir que maintenant désolé..

J'ai commencé le process de remplacer ce template par une CLI permettant entre autre d'initialiser un nouveau modèle Publicodes : publicodes/tools#50, ce qui permettra de plus facilement mettre à jour et corriger les bugs. Si tu as des idées n'hésites pas à me contacter que l'on puisse en discuter ensemble !

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.

2 participants