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

Relecture_FC #1476

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

Relecture_FC #1476

wants to merge 1 commit into from

Conversation

Miryad3108
Copy link
Contributor

No description provided.

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.

Merci beaucoup pour ta relecture ! juste une mini demande de changement sur le titre de la liste des API

@@ -157,17 +157,17 @@ fr:
Par ailleurs, toutes les API du bouquet API Particulier ne bénéficient pas de la modalité d'appel FranceConnect :

{:.fr-h6}
### <a name="liste-api-particulier-franceconnectees"></a>Liste des API FranceConnectées
### <a name="liste-api-particulier-franceconnectees"></a>Liste des API : API FranceConnectées et non FranceConnectées
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
### <a name="liste-api-particulier-franceconnectees"></a>Liste des API : API FranceConnectées et non FranceConnectées
### <a name="liste-api-particulier-franceconnectees"></a>Liste des API FranceConnectées

@Miryad3108 j'aurai gardé la première formulation car + percutant et + court.

Copy link
Contributor Author

@Miryad3108 Miryad3108 Feb 27, 2024

Choose a reason for hiding this comment

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

En lisant le titre, je m'attends à ne voir que des API FC du coup ça m'a assez étonné d'en voir des non FC dans la liste, d'où la proposition de modification. Mais on peut garder comme ça oui

Copy link
Contributor

Choose a reason for hiding this comment

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

On peut peut-être ajouter une petite phrase intro plutôt que de modifier le titre.

@@ -157,17 +157,17 @@ fr:
Par ailleurs, toutes les API du bouquet API Particulier ne bénéficient pas de la modalité d'appel FranceConnect :

{:.fr-h6}
### <a name="liste-api-particulier-franceconnectees"></a>Liste des API FranceConnectées
### <a name="liste-api-particulier-franceconnectees"></a>Liste des API : API FranceConnectées et non FranceConnectées
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
### <a name="liste-api-particulier-franceconnectees"></a>Liste des API : API FranceConnectées et non FranceConnectées
### <a name="liste-api-particulier-franceconnectees"></a>Liste des API : Liste des API FranceConnectées
Voici la liste des API du bouquet API Particulier et leurs modalités d'appel :

Copy link
Contributor

Choose a reason for hiding this comment

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

@Miryad3108 qu'est ce que tu en penses ? :)

@Samuelfaure
Copy link
Contributor

Vous voulez que je fix les conflits + merge? @Miryad3108 @DorineLam

@DorineLam
Copy link
Contributor

Vous voulez que je fix les conflits + merge? @Miryad3108 @DorineLam

Je laisse @Miryad3108 répondre :)

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