Skip to content

Refonte - Événements > Inscriptions - Créer/Modifier/Supprimer#2136

Open
stakovicz wants to merge 4 commits intoafup:masterfrom
stakovicz:feat-admin-event-tickets-edit
Open

Refonte - Événements > Inscriptions - Créer/Modifier/Supprimer#2136
stakovicz wants to merge 4 commits intoafup:masterfrom
stakovicz:feat-admin-event-tickets-edit

Conversation

@stakovicz
Copy link
Contributor

@stakovicz stakovicz commented Feb 2, 2026

Refonte de la création/modification et suppression d'une inscription.

Issue : #2125

@stakovicz stakovicz self-assigned this Feb 2, 2026
@stakovicz stakovicz force-pushed the feat-admin-event-tickets-edit branch 7 times, most recently from 766cc7d to 64a0362 Compare February 3, 2026 21:11
@stakovicz stakovicz requested review from Mopolo, agallou and vgreb February 4, 2026 12:08
@stakovicz stakovicz marked this pull request as ready for review February 4, 2026 12:08
@stakovicz stakovicz marked this pull request as draft February 4, 2026 12:19
@stakovicz stakovicz marked this pull request as ready for review February 5, 2026 19:17
@stakovicz
Copy link
Contributor Author

@agallou et @Mopolo je n'ai pas repris le champ "coupon" et "newsletter_nexen".

Vous en pensez quoi ?

@stakovicz stakovicz force-pushed the feat-admin-event-tickets-edit branch from 64a0362 to 1b97590 Compare February 20, 2026 21:04
@Mopolo
Copy link
Contributor

Mopolo commented Feb 22, 2026

@agallou et @Mopolo je n'ai pas repris le champ "coupon" et "newsletter_nexen".

Vous en pensez quoi ?

J'ai jamais vue de valeur dans le champ coupon et nexen je ne connaissais pas donc ça me parait ok à enlever. À voir ce qu'en dit @agallou

@stakovicz stakovicz force-pushed the feat-admin-event-tickets-edit branch from 1b97590 to 199a026 Compare February 28, 2026 07:27
Copy link
Contributor

@Mopolo Mopolo left a comment

Choose a reason for hiding this comment

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

Impec, une dernière remarque et tout le reste est bon pour moi :)

Comment on lines +25 to +27
{{ form_row(form.invoice.reference) }}
{{ form_row(form.invoice.authorization) }}
{{ form_row(form.invoice.transaction) }}
Copy link
Contributor

Choose a reason for hiding this comment

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

Je viens de faire quelques tests et quand on change une de ces 3 valeurs ça casse tout, et je pense que c'est normal que ça casse tout.

Je n'ai jamais eu à les modifier et je serais d'avis de les afficher mais de les rendre non-modifiables.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants