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

API-3494 Ajout du tag " nouveauté" l'API Cotisation CP et CI #1680

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

Conversation

Charlottecho
Copy link
Contributor

Ajout du tag " nouveauté".
@DorineLam : j'ai compris que je devais rajouter cette ligne "novelty: true" , mais je ne suis pas certaine de l'avoir ajoutée au bon endroit ( à la bonne ligne ) .

Ajout du tag " nouveauté"
Copy link

linear bot commented Sep 24, 2024

@DorineLam
Copy link
Contributor

ça fonctionne très bien !

image

Peut-être qu'on retire novelty à l'API liens de successions ?

Copy link
Contributor

@DorineLam DorineLam left a comment

Choose a reason for hiding this comment

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

ça fonctionne, tu peux merger. Peut-être un détail en plus si tu le souhaites : retirer le tag nouveau à certaines API pour mettre plus en valeur les "nouvelles nouveautés" ?

@Charlottecho
Copy link
Contributor Author

@DorineLam : plutot l'enlever à "Qualiopi" qui est plus ancienne que "Liens de succession" .. Ok pour toi ?

@DorineLam
Copy link
Contributor

@DorineLam : plutot l'enlever à "Qualiopi" qui est plus ancienne que "Liens de succession" .. Ok pour toi ?

yes ! Je te laisse faire la modif ? Tu me dis si tu t'en sors ? (Il faut que tu retrouves le fichier de qualiopi) et que tu t'assures de te mettre sur cette même branche ;)

@Charlottecho
Copy link
Contributor Author

@DorineLam quand tu dis "cette même branche" , c'est celle de "qualiopi" ou celle-ci que je viens de merger rdans ce thread ?

@DorineLam
Copy link
Contributor

@DorineLam quand tu dis "cette même branche" , c'est celle de "qualiopi" ou celle-ci que je viens de merger rdans ce thread ?

Coucou @Charlottecho, pour l'instant, tu n'as rien "mergé" (merger = Fusionner ta branche dans la branche principale appelée develop). Actuellement tu es donc sur la branche Charlottecho-patch-2. Ce que je te propose c'est de te rendre sur le code de la fiche CNETP, t'assurer que tu es toujours sur la branche Charlottecho-patch-2, faire tes changements, les valider, et ils s'ajouteront alors à ta branche Charlottecho-patch-2 que nous mergerons à la fin dans develop

@Un3x
Copy link
Contributor

Un3x commented Oct 30, 2024

A la limite ici, faite le en pair, parce que faut shipper !

@DorineLam
Copy link
Contributor

A la limite ici, faite le en pair, parce que faut shipper !

Je pense que ça a juste été oublié ^^ mais oui on va le faire à deux ;)

@Un3x
Copy link
Contributor

Un3x commented Dec 17, 2024

MAis shippons, c'est long là quand même.

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.

3 participants