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

Utilisation de la Gem dsfr-form_builder #5010

Open
wants to merge 1 commit into
base: production
Choose a base branch
from

Conversation

AntoineGirard
Copy link
Member

@AntoineGirard AntoineGirard commented Jan 23, 2025

Review app

Contexte

On a exporté le code de notre DSFR Form Builder dans une Gem séparée (https://github.com/betagouv/dsfr-form-builder) afin qu’elle soit utilisée et améliorée par la communauté. L’idée est donc de l’utiliser plutôt que de maintenir notre lib en local.

Solution

Étant donné que nous sommes au tout début de cette Gem, il peut y avoir des choses qui ne sont pas encore fixées et qui vont bouger. J’ai donc pris le parti de fixer la version de la Gem pour éviter une mise à jour qui contiendrait des breaking changes.
Une fois qu’on sortira la v1, nous pourrons être moins restrictifs sur le numéro de version.

@AntoineGirard AntoineGirard force-pushed the agd/use-dsfr-form-builder branch from 5b222fa to 202d20b Compare January 23, 2025 09:49
@AntoineGirard AntoineGirard marked this pull request as ready for review January 23, 2025 10:28
Copy link
Contributor

@adipasquale adipasquale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 testé sur la review app ! bravo pour l’extraction de la gem

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🚀 To deploy
Development

Successfully merging this pull request may close these issues.

2 participants