Skip to content

Remove duplicate tilmeldings mail call - #674

Draft
ThomasBow wants to merge 3 commits into
nextfrom
duplicate-mail-fix
Draft

ThomasBow wants to merge 3 commits into
nextfrom
duplicate-mail-fix

Conversation

@ThomasBow

Copy link
Copy Markdown
Contributor

Remove duplicate call as it gets called after being saved in signals.py

@krestenlaust

Copy link
Copy Markdown
Member

hmmmmmmmmmmmmmmmmm, i guess this could've been it all along. Was this introduced with my 'hotfix'?

Comment thread stregsystem/models.py Outdated
ThomasBow and others added 2 commits September 10, 2026 08:11
Co-authored-by: Kresten Laust <krestenlaust@gmail.com>

@krestenlaust krestenlaust left a comment

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.

Did you discover anything? The bug has cropped up again

@krestenlaust

Copy link
Copy Markdown
Member

I haven't been able to reproduce the bug locally

@ThomasBow

ThomasBow commented Sep 14, 2026 •

Copy link
Copy Markdown
Contributor Author

I too have not been able to reproduce it locally, so i was thinking we just make it a field on the member, wether they've been sent one or not. Or alternatively we should add some logging and wait till it happens again. I did remove something which seemed like it would cause a double call, but i don't see how it isn't happening every time

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