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

Mise à jour de descriptions d'aides #18

Merged
merged 12 commits into from
Nov 27, 2024

Conversation

Shamzic
Copy link
Contributor

@Shamzic Shamzic commented Nov 19, 2024

Dans l'esprit de #10, j'ai reformulé les descriptions qui dépassaient 420 caractères

Copy link

changeset-bot bot commented Nov 19, 2024

🦋 Changeset detected

Latest commit: 129e12c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@betagouv/aides-velo Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Collaborator

@EmileRolley EmileRolley left a comment

Choose a reason for hiding this comment

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

Merci @Shamzic pour les modifications. J'ai simplement trois remarques :

  1. Afin de raccourcir les descriptions je pense que l'on peut omettre de préciser le nom de la commune/CC/région dans la description car il figure déjà dans le titre de l'aide. J'ai proposé des modifications dans ce sens.
  2. Pour de prochaines contributions, tu peux directement faire une branche à partir du dépôt sans passer par un fork afin de pouvoir plus facilement itérer sur les modifications.
  3. La gestion des version et du changelog se fait automatiquement avec changesets, donc à la place de modifier manuellement le numéro de version du paquet, il suffit d'exécuter la commande npx changeset (voir la section Modification des aides)

Dis moi ce que tu en penses.

src/rules/aides.publicodes Outdated Show resolved Hide resolved
src/rules/aides.publicodes Outdated Show resolved Hide resolved
src/rules/aides.publicodes Outdated Show resolved Hide resolved
src/rules/aides.publicodes Outdated Show resolved Hide resolved
src/rules/aides.publicodes Outdated Show resolved Hide resolved
src/rules/aides.publicodes Outdated Show resolved Hide resolved
src/rules/aides.publicodes Outdated Show resolved Hide resolved
@Shamzic
Copy link
Contributor Author

Shamzic commented Nov 27, 2024

  1. Afin de raccourcir les descriptions je pense que l'on peut omettre de préciser le nom de la commune/CC/région dans la description car il figure déjà dans le titre de l'aide. J'ai proposé des modifications dans ce sens.

C'est une bonne suggestion, je suis aligné.

  1. Pour de prochaines contributions, tu peux directement faire une branche à partir du dépôt sans passer par un fork afin de pouvoir plus facilement itérer sur les modifications.

Ok, je n'ai pas pu faire une pull request directement sur une branche du répo car j'étais bloqué par une restriction (probablement à régler dans les paramètres).

  1. La gestion des version et du changelog se fait automatiquement avec changesets, donc à la place de modifier manuellement le numéro de version du paquet, il suffit d'exécuter la commande npx changeset (voir la section Modification des aides)

C'est noté 👍

@EmileRolley
Copy link
Collaborator

EmileRolley commented Nov 27, 2024

Ok, je n'ai pas pu faire une pull request directement sur une branche du répo car j'étais bloqué par une restriction (probablement à régler dans les paramètres).

J'ai rajouté les droits d'écriture à la team aides jeune 👍

@Shamzic Shamzic requested a review from EmileRolley November 27, 2024 13:57
@EmileRolley EmileRolley merged commit 44badf6 into betagouv:main Nov 27, 2024
2 checks passed
@github-actions github-actions bot mentioned this pull request Nov 27, 2024
@Shamzic Shamzic deleted the maj-descriptions-longues branch November 27, 2024 16:46
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