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

[Erreur contenu] Mise en exuergue API depréciée : Retire mention doc API Entreprise #1289

Merged
merged 2 commits into from
Nov 14, 2023

Conversation

DorineLam
Copy link
Contributor

Problème :

image

Ce que cette PR propose :

image

@DorineLam DorineLam self-assigned this Oct 20, 2023
@DorineLam DorineLam requested a review from Miryad3108 October 20, 2023 13:55
@skelz0r
Copy link
Member

skelz0r commented Oct 20, 2023

Je t'invite à revert les changements sur app/views/api_particulier/endpoints/show.html.erb qui n'ont pas lieu d'être
Et dans les wordings change directement la ligne, ça ne sert à rien de garder la ligne

@DorineLam
Copy link
Contributor Author

Je t'invite à revert les changements sur app/views/api_particulier/endpoints/show.html.erb qui n'ont pas lieu d'être Et dans les wordings change directement la ligne, ça ne sert à rien de garder la ligne

Je voulais garder le code en commentaire pour le remettre quand on sera passer en V3 ... C'est pas une bonne pratique ?

@skelz0r
Copy link
Member

skelz0r commented Oct 20, 2023

Je t'invite à revert les changements sur app/views/api_particulier/endpoints/show.html.erb qui n'ont pas lieu d'être Et dans les wordings change directement la ligne, ça ne sert à rien de garder la ligne

Je voulais garder le code en commentaire pour le remettre quand on sera passer en V3 ... C'est pas une bonne pratique ?

Pas sur des gestionnaires de versions. Tu peux revenir à n'importe quelle version avec git. Ici tu changes le texte et le jour où tu veux restaurer tu fais un revert de la modification. C'est 100x plus propre que de laisser des commentaires.

@DorineLam
Copy link
Contributor Author

Je t'invite à revert les changements sur app/views/api_particulier/endpoints/show.html.erb qui n'ont pas lieu d'être Et dans les wordings change directement la ligne, ça ne sert à rien de garder la ligne

Je voulais garder le code en commentaire pour le remettre quand on sera passer en V3 ... C'est pas une bonne pratique ?

Pas sur des gestionnaires de versions. Tu peux revenir à n'importe quelle version avec git. Ici tu changes le texte et le jour où tu veux restaurer tu fais un revert de la modification. C'est 100x plus propre que de laisser des commentaires.

Ok c'est pas facile pour moi la méthode que tu proposes car je ne sais pas faire.
Mais, je t'écoute et j'applique parce que tu sais mieux que moi ce qu'est un code propre ;)

@skelz0r
Copy link
Member

skelz0r commented Oct 20, 2023

Au pire tu regardes l'historique et tu c/c.

@skelz0r
Copy link
Member

skelz0r commented Nov 14, 2023

Tiens ça a trainé, je merge donc

@skelz0r skelz0r merged commit 0e22ef3 into develop Nov 14, 2023
3 of 4 checks passed
@skelz0r skelz0r deleted the fix-erreur-qfv1 branch November 14, 2023 11:12
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