-
Notifications
You must be signed in to change notification settings - Fork 2
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
email usability enhancements #318
Comments
Maybe a few too many things to pack into the subject line. |
Box 1 checked by #368 |
fine to present discovery report summary info in email body. fine to just put table contents in email for discovery report. might want to have some sort of cut off where we only present summary data for discovery reports over a certain size, since the reports can be huge. we won't be able to do this sort of thing for pre-assembly without tweaking the way it runs, since we don't collect reduced summary data on that, and we don't want to re-parse all the output everytime we want to send an email or display something. so pre-assembly email will probably be less informative than discovery report. |
Closed in favor of #976 |
feel free to break into smaller tickets if its easier to just implement one part at a time.
The text was updated successfully, but these errors were encountered: