-
Notifications
You must be signed in to change notification settings - Fork 5
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
How to link "matching" and "notification" workflow #115
Comments
This supersedes #95 , although maybe the answer is the same? |
I'd add one more user need:
(i.e. people might opt out) Otherwise, yes, I think the answer is the same. |
I think I'll implement previewing short-term as downloading, rather than trying to display it in the interface |
One more thing has struck me that I thought I'd flag. You'll recall that the system outputs 2 spreadsheets:
Each spreadsheet contains all of the participants, even if that participant has 0 matches. When we notify people of their matches, they get one of 4 email templates:
You may recall we discussed that I would manually remove the 'no matches' people from the CSV files to create the 4 CSV inputs needed for each of the 4 separate templates. This was a fudge at the time, but only took a few seconds. If we're automating the emails though, we need to go back and fix that problem; either by:
As ever, sorry for creating you more headaches! |
We will probably need to pop up a warning (something like 'you've got 10 minutes to check your file before it's deleted from the system')? |
Issue
The matching and notification workflows are currently completely separate.
User needs
Out of scope
The text was updated successfully, but these errors were encountered: