InvitationManager silently no-ops on non-invitable events/workshops while controllers flash success
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 72/100
Piste de recherche
Start at Admin::WorkshopsController#invite and Admin::EventsController#invite, then trace the three InvitationManager methods and their handle_asynchronously calls. Confirm that non-invitable targets produce a warning without enqueueing, while invitable targets retain the current success flow and email behavior.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Problem
Three InvitationManager methods guard with error strings — send_event_emails, send_workshop_emails, send_virtual_workshop_emails all return 'The X is not invitable' unless x.invitable?. All three run via handle_asynchronously, so the return value is discarded by the DelayedJob worker. The controllers (Admin::WorkshopsController#invite, Admin::EventsController#invite) ignore the return anyway and flash "Invitations will be emailed out soon." unconditionally.
Net effect: when the target isn't invitable, nothing is sent and nobody learns — the flash lies.
Proposed change
- Check
invitable?in the controllers before enqueueing and flash a warning instead of the success message when it's false. (Raising inside the async method can't fix the flash — the raise happens after the redirect.) - Remove the now-dead string returns from
InvitationManager; the internal batch-abort strings (start_invitation_batch→return result if result.is_a?(String)) may stay as internal control flow — implementer's call.
Acceptance
- Inviting to a non-invitable event/workshop shows a warning flash and enqueues nothing.
- Inviting to an invitable one behaves as today.
- Langage dominant
- Ruby
- Étoiles
- 104
- Forks
- 205
- Merge moyen
- 1 j 4 h
- PR mergées (30 j)
- 77
Préparer son environnement
- 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 codebar/planner
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 4/5 3-5 jours Accessibilité débutants 56/100
Les mainteneurs répondent en général sous 1 jour
-
refactoring tech debt
Difficulté 4/5 3-5 jours Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
refactoring tech debt
Difficulté 3/5 1-2 jours Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de codebar/planner
Issues similaires
-
Difficulté 1/5 1-3 heures Accessibilité débutants 78/100
TheOdinProject/curriculum#31433 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
stablyai/orca#24145 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
endoflife-date/endoflife.date#11194 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 1-3 heures Accessibilité débutants 84/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100