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

Arbre public cible api parteprise #1377

Merged
merged 2 commits into from
Sep 24, 2024
Merged

Conversation

DorineLam
Copy link
Contributor

@DorineLam DorineLam commented Sep 24, 2024

Un exemple de changement :

image

@DorineLam DorineLam requested a review from skelz0r September 24, 2024 07:21
@DorineLam DorineLam self-assigned this Sep 24, 2024
@skelz0r
Copy link
Contributor

skelz0r commented Sep 24, 2024

Je n'arrive pas à faire tourner en local la partie API Entreprise, mais je constate que ça passe tous les tests 🙄

@DorineLam
Copy link
Contributor Author

Je n'arrive pas à faire tourner en local la partie API Entreprise, mais je constate que ça passe tous les tests 🙄

Ah ... Je ne sais pas quoi faire du coup, de mon côté j'ai bien pu voir en local, par contre ça marche pas du tout en staging : https://staging.api.gouv.fr/les-api/api-particulier/demande-acces

Comment ferais tu ici ?

@skelz0r
Copy link
Contributor

skelz0r commented Sep 24, 2024

Comment ferais tu ici ?

j'en sais rien je pensais que ça marchait :')

@DorineLam
Copy link
Contributor Author

J'ai l'erreur suivante en local, sur la fiche API Entreprise, mais pas sur la page contenant l'arbre de décision de l'API ENtreprise :

image

@skelz0r
Copy link
Contributor

skelz0r commented Sep 24, 2024

L'erreur existe aussi sur master en fait, donc je ne pense pas que ça soit bloquant (suffit de cliquer sur la croix en haut à droite).

@DorineLam
Copy link
Contributor Author

Oui ok, du coup est-ce que je merge ? Parce que si cette erreur n'est pas bloquante, je pense donc que ma PR fonctionne vu que je la vois en local ?

@skelz0r
Copy link
Contributor

skelz0r commented Sep 24, 2024

Si t'es OK sur les wordings j'ai envie de dire #yolo

@DorineLam DorineLam enabled auto-merge September 24, 2024 16:47
@DorineLam
Copy link
Contributor Author

Si t'es OK sur les wordings j'ai envie de dire #yolo

Je veux bien que tu approuves, je ne peux pas merger

@DorineLam DorineLam merged commit 92d102e into master Sep 24, 2024
11 checks passed
@DorineLam DorineLam deleted the arbre-public-cible-api-parteprise branch September 24, 2024 16:52
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