-
Notifications
You must be signed in to change notification settings - Fork 71
Refonte de la class Branche #2017
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
Conversation
| </div> | ||
| <aside id="sidebar-article" class="mod item w33 m50 t100"> | ||
| <h2>L'afup organise...</h2> | ||
| {{ sidebar.naviguer(1, 2, "externe", "") }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je n'ai pas trouvé de cas d'usage pour la colonne de droite, est-ce que c'est toujours utilisé ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Peut-être que @agallou saura répondre
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TLDR : oui, c'est utilisé, mais c'est moche et serait à supprimer.
On retrouve cela sur la page de retour après paiement des cotisations.
Les feuilles affichées sur la home y sont affichées. Ça donne cela :

Où dans le renderer : https://localhost:9205/association/paybox-redirect?total=3000&cmd=C2025-201220252348-0-9-TEST-B6B&autorisation=XXXXXX&transaction=85846868&status=00000
on va appeler cette méthode après avoir appeller le header :
web/sources/Afup/Corporate/Page.php
Line 118 in ad4afa6
| $branche->navigation_avec_image(true); |
On y affiche deux feuilles :
- La Veille de l'AFUP
- AFUP Day 2026
Où avec le {{ sidebar.naviguer(1, 2, "externe", "") }} on va rechercher sur 2 niveaux les enfants de la feuille 1, la feuille colonne de droite :
web/sources/Afup/Corporate/Feuille.php
Line 11 in f2838fd
| public const ID_FEUILLE_COLONNE_DROITE = 1; |
Normalement ces items sont maintenant utilisés sur la home. Et leur rendu n'est pas du mieux sur la page cotisation (et ils arrivent après avoir payé, si on voudrait les afficher ça serait plus avant).
A terme ça serait bien qu'on revoie cette page pour avoir des actions à faire / un meilleure message selon si c'est notre première cotisation / un renouvellement / si on est une entreprise, mais en attendant ça sera une bonne chose si on enlève cela.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
J'ai retiré la sidebar et ouvert une issue pour retravailler les pages de retour du processus de paiement des cotisations
9dfcee8 to
e64869c
Compare
a62641e to
bcb0ba8
Compare
bcb0ba8 to
986e270
Compare
| private _Site_Base_De_Donnees $bdd; | ||
|
|
||
| public function __construct() | ||
| { | ||
| $this->bdd = new _Site_Base_De_Donnees(); | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ça fait plaisir de voir ça :D
986e270 to
ffd03f9
Compare
No description provided.