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
feat: remove non division members & inactive members no state change #2799
Conversation
[ci skip] [skip ci]
…blade.php Co-authored-by: Alex Toff <atoff@users.noreply.github.com>
[ci skip] [skip ci]
…:VATSIM-UK/core into remove-non-division-members-waiting-list
[ci skip] [skip ci]
app/Notifications/Training/RemovedFromWaitingListInactiveAccount.php
Outdated
Show resolved
Hide resolved
app/Notifications/Training/RemovedFromWaitingListNonHomeMember.php
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we just double check the consistency on the debug logs between the two listeners (e.g. the text that is logged and at one stage).
Otherwise I'm happy.
Good shout. Amended to have the best level of detail across the board. |
resources/views/emails/training/waiting_list_non_home_removal.blade.php
Outdated
Show resolved
Hide resolved
…blade.php Co-authored-by: Alex Toff <atoff@users.noreply.github.com>
Adds listeners to
AccountAltered
event, which, when triggered, will remove:-1
internally.When the removal happens, they will be sent an email depending on which condition they have no longer met to be on the waiting list.