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

Bug/points exceeding limit #243

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

hericlibong
Copy link

Description de la Pull Request

# Correction du Problème de Dépassement des Points Disponibles

Cette pull request apporte une correction au problème de dépassement des points disponibles lors de la réservation de places pour une compétition, conformément à l'issue [#226](#).

## Modifications Principales

- **Validation des Points Disponibles** : Avant de confirmer une réservation, le code vérifie que le nombre de points du club est suffisant pour couvrir le nombre de places demandées.
- **Validation des Places Disponibles** : La réservation est également bloquée si le nombre de places demandées excède le nombre de places encore disponibles pour la compétition.
- **Messages d'Erreur** : Un message flash informe l’utilisateur en cas d'échec de la réservation, indiquant soit l’insuffisance des points, soit le manque de places disponibles.
- **Mise à Jour des Données** : Si toutes les conditions sont remplies, la réservation est validée et le nombre de places et de points est mis à jour en conséquence.

## Tests Effectués

- **Tentative de réservation avec des points insuffisants** : le message d'erreur apparaît, la réservation est bloquée.
- **Tentative de réservation avec des places insuffisantes** : le message d'erreur apparaît, la réservation est bloquée.
- **Réservation réussie avec points et places disponibles suffisants** : le message de confirmation s'affiche, et les points et places sont mis à jour.

## Références

Cette PR se réfère à l’issue [#226](#).

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.

1 participant