Skip to content

Conversation

@jiordiviera
Copy link

…e la page, et aussi ajout d'une image de secours lorsque l'image est inexistant

…e la page, et aussi ajout d'une image de secours lorsque l'image est inexistant
@jiordiviera jiordiviera changed the base branch from main to develop October 26, 2024 06:00
@jiordiviera jiordiviera changed the base branch from develop to main October 26, 2024 06:00
@jiordiviera
Copy link
Author

Correction de l'incrémentation des vues d'un post à l'actualisation de la page, et aussi ajout d'une image de secours lorsque l'image est inexistant.

@mckenziearts
Copy link
Member

Merci pour ta PR @jiordiviera mais je ne peux pas la merge parce qu'elle contient beaucoup trop de bruit. Et pour faciliter les reviews c'est une feature par PR. Les PRs doivent être le plus léger possible, et dans ta PR y'a des modifications de fichier qui sont effectuées qui n'ont rien avec la fonctionnalité.

Ce que je te propose de faire

  • Si tu utilises vscode, désactive le format on save. Je suppose que c'est lui qui modifie l'architecture du HTML
  • Faire des PRs qui font uniquement ce que tu décris
  • Run les tests composer test:pest avant de pousser pour être sur que tu n'as rien cassé et composer lint pour fixer le style dans les classes

Pour le moment je vais fermer cette PR. le temps que tu proposes une qui respecte ce que je viens de te dire

Merci.

@jiordiviera jiordiviera deleted the fix-post-views_post-image branch October 26, 2024 14:49
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