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

Petites amélioration de la documentation de l'api #4815

Merged
merged 4 commits into from
Nov 19, 2024

Conversation

victormours
Copy link
Contributor

Contexte

La nouvelle homepage sur rdv.anct.gouv.fr met en avant la documentation de notre API, et les discussions avec Démarches Simplifiées et Mon Suivi Social nous amènent à leur passer des liens vers notre documentation.

Le problème, c'est que ça fait longtemps qu'on ne s'était pas trop penché dessus, et il y a pas mal d'inexactitudes. Le but de cette PR est donc de passer un petit coup de frais sur cette documentation.

Certains passages ont été écrits par des devs qui étaient juste de passage, et qui avait une vision pour cette api qui ne s'est pas concrétisée.

Cette PR n'est pas une nouvelle version parfaite, mais elle est une amélioration raisonnable pour un vendredi de cooldown.

- `endpoint` est le nom du point de terminaison

Par exemple, on aura : `https://<domain>/api/v1/absences`

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ce passage est un bon exemple de description très longue pour dire des choses très simples : c'est l'api v1, et les routes sont explicites dans tout le reste de la doc

@@ -1,10 +1,9 @@
L'API de RDV-Solidarités vous permet de lire des données dans notre base depuis votre logiciel.
L'API de RDV Service Public permet à nos partenaires, comme par exemple Démarches Simplifiées et RDV Insertion,d'ajouter de la gestion de rendez-vous dans leurs applications métier.
Elle peut aussi être utilisée pour synchroniser les calendriers des agents de votre administration avec ceux qu'ils ont sur leur compte RDV Service Public.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

La doc de l'api étant visible par des partenaires potentiels, c'est bien qu'ils puissent se projeter un peu plus sur ce qu'elle permet de faire.


# Dépréciations
Copy link
Contributor Author

Choose a reason for hiding this comment

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

j'ai déplacé cette section à la fin de la doc. Ça fait longtemps que cette dépréciation existe, et c'est sans doute pas ce qu'on veut mettre en avant pour nos nouveaux partenaires

Pour la version production, les requêtes doivent être adressées à https://www.rdv-solidarites.fr et non à https://rdv-solidarites.fr.

Pour la version démo, les requêtes doivent être adressées à https://demo.rdv-solidarites.fr.

# Authentification

Certains points de terminaison sont réservés aux agents authentifiés, dans la limite de leur rôle au sein de l'application.
Presque tous les points de terminaison sont réservés aux agents authentifiés, dans la limite de leur rôle au sein de l'application.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

un seul endpoint ne nécessite pas d'authentification (public_links)

target_url: target_url,
secret: secret,
subscriptions: %w[rdv user user_profile organisation motif lieu agent agent_role]
)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

c'était une bonne occasion de faire tourner un seul test (à la place de 5), mais de garder le même niveau de détail des messages d'erreur avec ce matcher (plus de signal et moins de bruit dans le fichier de test, et ça permet de générer la doc plus rapidement)

@@ -52,7 +52,7 @@
"v1/api.json" => {
openapi: "3.0.1",
info: {
title: "API RDV Solidarités",
title: "API de RDV Service Public",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Pour les partenaires qui arrivent depuis la landing page de RDV Service Public (qui est en plus notre marque principale), c'est quand même mieux de voir ce titre

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.

merci d’avoir mis à jour tout ça 👍

spec/swagger_helper.rb Outdated Show resolved Hide resolved
spec/swagger_helper.rb Outdated Show resolved Hide resolved
@victormours victormours enabled auto-merge (squash) November 19, 2024 08:45
@victormours victormours merged commit 3821418 into production Nov 19, 2024
11 checks passed
@victormours victormours deleted the small-api-simplifications branch November 19, 2024 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

2 participants