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

Corrigé l'erreur avec intervention_count #24

Merged
merged 5 commits into from
Apr 6, 2023

Conversation

ClementCadieux
Copy link
Contributor

intervention_count réduisait de 1 au lieu d'augmenter.

Copy link
Collaborator

@p2xcode p2xcode left a comment

Choose a reason for hiding this comment

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

Je ne suis pas certain que ce soit ça…. Ce que ça fait ça c’est que si une intervention dans un debat existe déjà dans le hub, il skip, où il l’écrase. S’il skip, il ne fait pas qu’il compte cette intervention comme une intervention ajoutée au hub.

@ClementCadieux
Copy link
Contributor Author

Je ne suis pas certain que ce soit ça…. Ce que ça fait ça c’est que si une intervention dans un debat existe déjà dans le hub, il skip, où il l’écrase. S’il skip, il ne fait pas qu’il compte cette intervention comme une intervention ajoutée au hub.

Oui je viens de mieux comprendre ça. Je fais d'autres tests, mais je crois quand même que cette ligne là causait un problème. Je test autre chose en ce moment.

Copy link
Collaborator

@p2xcode p2xcode left a comment

Choose a reason for hiding this comment

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

C'est intéressant comme solution.

@p2xcode
Copy link
Collaborator

p2xcode commented Apr 4, 2023

C'est intéressant comme solution.

Je te laisse merger @ClementCadieux

@ClementCadieux ClementCadieux merged commit f9f4ca1 into main Apr 6, 2023
@p2xcode p2xcode deleted the fixLogsQuebecDebates branch June 27, 2023 22:47
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