-
Notifications
You must be signed in to change notification settings - Fork 34
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
On-demand report generation #223
Comments
Update - 05/05/2023The Summary page is very similar to what we planned for option F3.) meaning that we can hitch the backend for the on-demand report feature to the Summary frontend. I will keep the rest of the plan intact (shown below) for posterity, but it is likely that all we will need to do is stitch the On-Demand Report backend to Summary's frontend. The above tasks can be broken-down into sub-tasks:
All of these functions will be very similar to one another, so I see the frontend and backend implementation going two ways. Possible backend implementations:B1.) Filtering the results from sub-task 1a. to generate different final reports. Possible frontend implementations:F1.) Creating a 'Report' page that is updated with the results found from the filtered/separate functions. Thoughts:Personally, I am leaning towards B2. and F2. and then building the functionality from there. What are other's thoughts on this? I'll start working on a design tomorrow depending on what people think. |
@jayrevolinskyjr I think building B2 and F2 first sounds reasonable. We can always add new features on the FE like charts and visualizations later. |
Looks like there is a massive amount of overlap between this issue and #152. So, option F3 seems mostly finished, all that really needs to be done is update/implement the backend. Updating and implementing the frontend can be a separate issue with it's own tasks. |
Based on the original requirements #65
Implement on-demand report generation for:
The text was updated successfully, but these errors were encountered: