People enjoy the instant gratification of a random alert.
This brings back email notifications to organizers on volunteer sign-up so that relevant people can react as soon as possible.
The email now will contain anonymous data (team name and t-shirt size) and a link for curious people to view the profile.
I have considered adding another email address for such notifications, but it seems unnecessarily complicated.
Proof of work:
![image](/attachments/dabd0375-17ac-43c8-9698-678e768aa111)
![image](/attachments/fed16195-e668-47c7-9647-5fe069f1ed2d)
Reviewed-on: #48
Co-authored-by: Tocho Tochev <tocho@tochev.net>
Co-committed-by: Tocho Tochev <tocho@tochev.net>
Before this change the count of volunteers in a team was not in sync with the filters applied.
![image](/attachments/0a3a0c5c-f3c2-4ef4-89d7-0bcf1fb9302d)
After the change:
![image](/attachments/b88719f6-fadf-4f07-b4d4-985533b0ea2f)
BTW I feel that we will end up refactoring the relation at some point in the future.
Reviewed-on: #46
Co-authored-by: Tocho Tochev <tocho@tochev.net>
Co-committed-by: Tocho Tochev <tocho@tochev.net>
We want to have some indication for that some volunteers haven't confirmed their email.
Perhaps "unverified" is bad terminology, but until we have proper "verification" it will suffice.
(Ideally there would also be a filter, but my rails is way too rusty...)
Reviewed-on: #45
Co-authored-by: Tocho Tochev <tocho@tochev.net>
Co-committed-by: Tocho Tochev <tocho@tochev.net>
I don't think the `.docker-ignore` does anything when there is a `.dockerignore` so let's remove it.
I'm open to amending the PR.
Reviewed-on: #43
Co-authored-by: Tocho Tochev <tocho@tochev.net>
Co-committed-by: Tocho Tochev <tocho@tochev.net>
Before this change if someone is using `localhost` for development instead of `127.0.0.1`, he would get a missing template error.
Reviewed-on: #44
Co-authored-by: Tocho Tochev <tocho@tochev.net>
Co-committed-by: Tocho Tochev <tocho@tochev.net>
This is a docker-compose based on production config, without mounting of the local folders, which I admit is a strange setup, but this was the simplest for me to get the local development going.
I'm perfectly ok with not merging this PR.
Reviewed-on: #47
Co-authored-by: Tocho Tochev <tocho@tochev.net>
Co-committed-by: Tocho Tochev <tocho@tochev.net>
I will raise the changes to the upstream project after review https://github.com/tochev/devise-i18n/pull/1/files
Co-authored-by: Albert Stefanov <aastefanov@outlook.com>
Reviewed-on: #49
Co-authored-by: Tocho Tochev <tocho@tochev.net>
Co-committed-by: Tocho Tochev <tocho@tochev.net>