Stop assuming all upperclassmen are on every packet
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 52/100
Piste de recherche
Commencez par packet/templates/upperclassmen_totals.html à la ligne 39 et packet/routes/upperclassmen.py aux lignes 51-62. Suivez la manière dont les données des upperclassmen atteignent la section active, puis assurez-vous que le dénominateur affiché ne compte que les paquets que chaque upperclassman peut signer. C’est terminé lorsque les décomptes actifs utilisent num_can_sign_as_upper au lieu de num_total lorsque l’état d’adhésion exclut les paquets précédents.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
This line makes the assumption that every single upperclassman is a required signature on every packet. This may not be the case, due to e.g. changes in membership state that don't influence earlier packets. This results in a confusing list of signature counts when an upperclassman is required on only some of the packets, since they are listed as $num_signed / $num_total in the active section. This aught to be $num_signed / $num_can_sign_as_upper
https://github.com/ComputerScienceHouse/packet/blob/f61cd2b5b616e3b7f35fad95b515313ea2d50bcb/packet/templates/upperclassmen_totals.html#L39
This will probably require injecting more data into the upperclassmen dict here so that it's accessible in the template.
https://github.com/ComputerScienceHouse/packet/blob/f61cd2b5b616e3b7f35fad95b515313ea2d50bcb/packet/routes/upperclassmen.py#L51-L62
- Langage dominant
- Python
- Étoiles
- 25
- Forks
- 33
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
Lance le conteneur de développement du projet dans votre navigateur, avec votre propre compte GitHub.
- Fournit un Dockerfile ou un fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de ComputerScienceHouse/packet
-
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
ComputerScienceHouse/packet#413 ·
-
Upgrade to bootstrap 5Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 38/100
ComputerScienceHouse/packet#411 ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 30/100
ComputerScienceHouse/packet#408 ·
-
React Frontend?Ouverteux enhancement
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
ComputerScienceHouse/packet#404 ·
-
ux enhancement
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
ComputerScienceHouse/packet#403 ·
Toutes les issues de ComputerScienceHouse/packet
Issues similaires
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
awslabs/visual-asset-management-system#414 ·
Les mainteneurs répondent en général sous 1 jour
-
bug v1 v2
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
modelcontextprotocol/python-sdk#3670 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
aicell-lab/bioengine#232 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
modelscope/evalscope#1836 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100