-
Notifications
You must be signed in to change notification settings - Fork 209
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
Suggestion/Feature Request: Export reports #1588
Comments
I would like help developing this issue. will submit a first draft of approach and downloaded report (in HTML). From there we can confirm what details needs to be modify. |
Yes, let's do a very minimal (3-4 point) featureset and then we go from there once it's reviewed. Basic idea is that, there will be a button (location TBD) and when the admin clicks on it, he'll see an HTML or PDF version of the report. |
First Draft: We can download report in pdf or html. User can choose report file type with dropdown. Currently, the position of download button is beside configure button. Report is generated based on selected date range. Report contains: Active time, Last Checked, Last Response Time, Avg Response Time, Uptime, DownTime and History. There is issue with theme for report, trying to fix it that's why it took time for First Draft. still working on it. video of feature: FirstDraftReportGenerator.mp4@gorkem-bwl @ajhollid can you look and tell the changes needed to be done before first of many pull request. |
Thanks for this. Let me come up with a template and what to include in it, and we go from there! |
Just fyi, I didn't have much time to think about it, but I'll comment once as soon as possible! |
Thanks @DMJain. AFAIK this is a rendered / simplified version of the page. Can you confirm? |
Hey @gorkem-bwl, currently yes a component is rendered with MUI and than being converted into html content before downloading. But I'm thinking of not using MUI for Report Page. instead using simple HTML & CSS. Not able to use Theme provider as component is not being mounted to actual DOM, due to this predefined theme are not getting applied properly to report. |
Is it possible to get this information via APIs? The reason is that the screen layout is not proper, and info presented there is not the same with what is explained in the issue. |
Hey, here is the new pdf report with only detail which are explained in issue. Portfolio-monitor-report-2025-01-26.pdf I have tried matching the pdf with style of website using MUI but theme is still an issue as report is not wrapped inside ThemeProvider. so using HTML-CSS. Waiting for layout to apply to report. @gorkem-bwl what changes are needed to be made? |
You are right on track. I checked the PDF file and let me give this a thought today and come back to you. This style is more than enough for now - no need to work on styling (e.g styling can be applied at a later date). |
Hey @gorkem-bwl , Any updates on the PDF file and the content that needs to be added? Regarding the custom date range, it's difficult to update with the current implementation. Right now, the date range is based on day, month, and year, and the start and end dates are calculated on the backend. Because of this, there's no API endpoint where I can send a custom end date to generate a report. Modifying the current implementation would be costly. A faster approach could be creating a new endpoint, but it requires proper design, review, implementation, and testing. Could you look into this? For now, we can proceed with the current implementation (day, month, year) and consider this change in future updates. Thanks! |
Sorry - dropped the ball on this. A few modifications I would propose:
Regarding the range, what is the current date range you have in the sample report? |
@DMJain how long does it take for the server to create a report? |
Something in HTML would be more interesting, styled to match the site's own design, with options for dark mode or light mode, and including some key data such as:
With these data points, averages could also be generated, as already available for daily, weekly, and monthly periods. A fourth and final option could calculate averages from the date the site was added up to the date the report is generated.
You could use the style of this site but add the features mentioned earlier. I believe this would make it much more complete. You could even use this idea and set it as the default option. However, an HTML report with charts and numerical data would be perfect.
The text was updated successfully, but these errors were encountered: