Skip to content

Admin: Configuration de la page "devenir sponsor" - #2445

Open
vgreb wants to merge 1 commit into
afup:masterfrom
vgreb:feature/admin-become-sponsor
Open

vgreb wants to merge 1 commit into
afup:masterfrom
vgreb:feature/admin-become-sponsor

Conversation

@vgreb

@vgreb vgreb commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Suite de #2437 et #2438 : ajoute une page d'administration (Évènements > Page devenir sponsor, /admin/event/become-sponsor) pour piloter /become-sponsor sans déploiement. On y choisit le mode : redirection vers la page sponsoring du dernier évènement, ou page de sponsoring de l'AFUP sans évènement. On y édite aussi les textes FR/EN en markdown et on y dépose les dossiers PDF FR/EN (htdocs/docs/pack-sponsoring-afup-{fr,en}.pdf).

Les réglages sont stockés dans une nouvelle table à ligne unique, afup_sponsorship_settings. La migration l'initialise en mode sans évènement avec les textes actuels, ce qui conserve le comportement de la prod. La clé de traduction become_sponsor.generic.text est supprimée. BecomeSponsorLatestAction, qui n'avait plus de route, est supprimé aussi : sa logique de redirection passe dans BecomeSponsorGenericAction. Si le dossier de la langue demandée manque, celui de l'autre langue est envoyé. En mode évènement, s'il n'existe aucun évènement, la page sans évènement s'affiche.

@vgreb
vgreb marked this pull request as ready for review November 4, 2026 19:22
@vgreb
vgreb force-pushed the feature/admin-become-sponsor branch from b35fd40 to 21efcae Compare November 7, 2026 23:02
*/
#[ORM\Entity(repositoryClass: SponsorshipSettingsRepository::class)]
#[ORM\Table(name: 'afup_sponsorship_settings')]
class SponsorshipSettings

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

On ne devrait pas faire des class en français ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

On remomme en ParametrageSponsoring ?

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