Skip to content
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

Send email for invitationer, nu også med ekstra tekst #956

Conversation

mhewel
Copy link
Contributor

@mhewel mhewel commented Aug 22, 2023

No description provided.

@what-the-diff
Copy link

what-the-diff bot commented Sep 24, 2023

PR Summary

  • Enhanced Email Template Handling
    This update introduces additional support for email templates in the members-related admin actions. This will allow admin users to handle email templates within the admin interface more efficiently.

  • New Field Addition in Mass Invitation Form
    A new form field, named 'email_text', has been added to the 'MassInvitationForm'. This allows the selection of invitation emails when mass-inviting users.

  • Email Invitation Logic Updates
    The logic for sending email invitations has been adjusted. This logic, previously part of the 'ActivityInvite.save' method, has now been relocated to a new method. This change should result in a more efficient process for managing activity invitation actions.

  • Added Email Template Section in Admin Template
    A new section has been added to the 'invite_many_to_activity' admin template. This section provides details about the email template, enhancing visibility and management of email templates.

  • Updated Test Case for 'email_text' Field
    The testing setup for managing the 'email_text' field in mass invite actions has been updated. This ensures the new changes remain compatible and function as expected under test conditions.

Copy link
Contributor

@rasmusselsmark rasmusselsmark left a comment

Choose a reason for hiding this comment

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

Ser korrekt ud, vi lægger på staging og tester

@rasmusselsmark rasmusselsmark merged commit 63ea84b into master Sep 24, 2023
2 checks passed
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.

[Feature]Email ifm invitation skal være flexibel ligesom ved sletning fra venteliste
2 participants